-
-
Notifications
You must be signed in to change notification settings - Fork 278
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
Stop using newly added cops #1065
Conversation
lib/rubocop/cop/rspec/multiple_expectations.rb:73:32: W: Lint/ToEnumArguments: Ensure you correctly provided all the arguments. expectations_count = to_enum(:find_expectation, node).count ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ We'd still love to use OUR new cops while working on our codebase, let's see how this option works when set under RSpec namespace when we introduce the first pending cop.
@bquorning Merge at will. |
I ran into this again recently. I think it’s actually a bug – see also rubocop/rubocop#8941 (comment) |
It doesn’t work.
|
What if we define This could fix the error, but I'm still uncertain if it would work as intended. Another concern is if changing the root |
Nope, that doesn’t work :sad_panda: |
This is quite frustrating, we can't eat our own dogfood 😄 |
(source)
We'd still love to use OUR new cops while working on our codebase, let's see how this option works when set under RSpec namespace when we introduce the first pending cop.
Before submitting the PR make sure the following are checked:
master
(if not - rebase it).CHANGELOG.md
if the new code introduces user-observable changes.bundle exec rake
) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).