Skip to content
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

AllCops:EnabledByDefault #4005

Closed
betesh opened this issue Feb 2, 2017 · 3 comments
Closed

AllCops:EnabledByDefault #4005

betesh opened this issue Feb 2, 2017 · 3 comments

Comments

@betesh
Copy link
Contributor

betesh commented Feb 2, 2017

Rubocop should have an EnabledByDefault option, just like there's a DisabledByDefault option.

If I want to Enable all cops, I need to read through all the docs to figure out which ones are disabled by default. It would be far more efficient if I could simply configure Rubocop to enable all of them.

Also, even if I've already enabled all cops that are disabled by default, if a new cop is added in a later version, I need to read through the Change Log to know about it.

@bbatsov
Copy link
Collaborator

bbatsov commented Feb 3, 2017

Sounds like a reasonable idea to me.

@betesh
Copy link
Contributor Author

betesh commented Feb 3, 2017

Great, I'm working on a PR

@betesh
Copy link
Contributor Author

betesh commented Feb 3, 2017

@bbatsov Can you take a look at the failing test on the PR (#4014)? I think it's a bug in the existing code but I'm not sure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants