New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add default configuration file for all the rules #897
Comments
should produce:
For 2), it's not always easily possible to reuse default value - since default is sometimes already parsed value. We could add something like "default_config_value" or "raw_default" for such cases.
|
|
With such requirements I'm thinking it should be better to not overcomplicate things and do not create fully automatic solution but semi one. For example we can have class for producing the configuration files with some options manually added in the code and some options (like included rules) could be auto-generated. As opposed, we could refactor our config class a bit to have something along: Options = [ And then use such list to create argparse and toml parsers, and use it for generating the default config. |
|
--ext-rules accepts string with comma delimited names. While self.ext_rulesis already parsed container (empty set). For such case I will display some example value, ie
I will try to create some POC because it could clear out some problems if I will be able to achieve it easily. |
I have managed to implement POC that reuses existing code for creating argument parser (by moving it to separate class). Thanks for adding few extra arguments like whether given option should be included in default config/commented out/default value overwritten I have produced:
The next step would be extending include with commented rule names, ie:
|
To make it easier for the users to enable/disable selected rules and other CLI options, we can provide a default configuration file that will also contain all community rules (marked as disabled) and a comment next to each option explaining briefly what it does.
Something like this
.pylintrc
file here.That way, people would not need to go through documentation of all the rules, they will just quickly skim through the configuration file, and they should at least has some basic understanding if they need it.
We can also add some
robocop init
command that will produce such file.This task is part of the #809 issue.
The text was updated successfully, but these errors were encountered: