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

-auto-gen-config and --rails are order sensitive #2194

Closed
agrimm opened this issue Aug 27, 2015 · 1 comment
Closed

-auto-gen-config and --rails are order sensitive #2194

agrimm opened this issue Aug 27, 2015 · 1 comment

Comments

@agrimm
Copy link
Contributor

agrimm commented Aug 27, 2015

I can do

bundle exec rubocop --rails --auto-gen-config

but not

bundle exec rubocop --auto-gen-config --rails

getting the error message

--auto-gen-config can only be combined with --exclude-limit.

There may be a good reason to only allow --rails before --auto-gen-config, rather than afterwards, but as it's currently written, people may think that it's just not possible to use --rails at all.

@jonas054
Copy link
Collaborator

I think I was the one who added the validation for --auto-gen-config a long time ago, and I obviously made a mistake. This might be a good opportunity to improve it and not be more strict that we need to. What's wrong with combining --auto-gen-config with other options really? How about assuming that people know what they're doing instead?

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

Successfully merging a pull request may close this issue.

2 participants