-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
false positive for Style/Mixin #5261
Labels
Comments
Definitely, the |
Thanks for the feedback. I confirmed this reproduction. I'm thinking about opening a PR for this issue later. |
8 tasks
koic
added a commit
to koic/rubocop
that referenced
this issue
Dec 19, 2017
…ng inside module Fixes rubocop#5261. Correspondence to use case for `include` belonging to module was insufficient. This PR added a reproduction test and fixed it.
bbatsov
pushed a commit
that referenced
this issue
Dec 20, 2017
…de module Fixes #5261. Correspondence to use case for `include` belonging to module was insufficient. This PR added a reproduction test and fixed it.
This was referenced Dec 27, 2017
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I get a complaint for the following code:
"Style/MixinUsage: include is used at the top level. Use inside class or module"
No, it's not used at the top level.
The text was updated successfully, but these errors were encountered: