Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Default config values should not require entry in config file #404

Closed
vigyasharma opened this issue Aug 31, 2020 · 0 comments · Fixed by #402
Closed

Default config values should not require entry in config file #404

vigyasharma opened this issue Aug 31, 2020 · 0 comments · Fixed by #402

Comments

@vigyasharma
Copy link
Contributor

In general, if a config value does not need to be overridden, we should be able to skip it from config files and directly use code defaults.
The following configurations in code however, require an empty list to be provided, because of the way they are initialized. We should change this and give these configs, empty lists as default values.

  "muted-rcas": [],
  "muted-deciders": [],
  "muted-actions": [],
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant