You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
If the plugins attribute is missing from safeeyes.json, safeeyes crashes with
Traceback (most recent call last):
File "/nix/store/…-safeeyes-2.1.6/bin/.safeeyes-wrapped", line 9, in <module>
sys.exit(main())
^^^^^^
File "/nix/store/…-safeeyes-2.1.6/lib/python3.11/site-packages/safeeyes/__main__.py", line 116, in main
config = Config()
^^^^^^^^
File "/nix/store/…-safeeyes-2.1.6/lib/python3.11/site-packages/safeeyes/model.py", line 327, in __init__
utility.merge_plugins(self.__user_config)
File "/nix/store/…-safeeyes-2.1.6/lib/python3.11/site-packages/safeeyes/utility.py", line 595, in merge_plugins
for plugin in config['plugins']:
~~~~~~^^^^^^^^^^^
KeyError: 'plugins'
I think safeeyes could have a default value for this attribute so as not to require its presence in the configuration file. This way, users can keep their configuration file as small as possible, containing only what is strictly different from the default.
In the same spirit, it would be much better if the user could override 1 plugin without specifying all the non-changed ones as well. So, the following configuration file:
Describe the bug
If the
plugins
attribute is missing fromsafeeyes.json
, safeeyes crashes withI think safeeyes could have a default value for this attribute so as not to require its presence in the configuration file. This way, users can keep their configuration file as small as possible, containing only what is strictly different from the default.
In the same spirit, it would be much better if the user could override 1 plugin without specifying all the non-changed ones as well. So, the following configuration file:
should be a valid way of saying
To Reproduce
Steps to reproduce the behavior:
safeeyes.json
plugins
optionExpected behavior
It should be possible to both:
safeeyes.json
and still get a working safeeyesDesktop (please complete the following information):
The text was updated successfully, but these errors were encountered: