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

Jump to bottom

Lint/SafeNavigationChain should allow in? (ActiveSupport) method #8500

Closed
johnnyshields opened this issue Aug 9, 2020 · 1 comment · Fixed by #8662
Closed

Lint/SafeNavigationChain should allow in? (ActiveSupport) method #8500

johnnyshields opened this issue Aug 9, 2020 · 1 comment · Fixed by #8662
Labels
enhancement

Comments

@johnnyshields
Copy link

If ActiveSupport is required, the .in? method should be permitted in a safe-navigation chain:

foo&.to_sym.in?([:baz, :qux])

Currently this raises the error: Lint/SafeNavigationChain: Do not chain ordinary method call after safe navigation operator.

RuboCop version: 0.89.0
@tejasbubane tejasbubane mentioned this issue Sep 7, 2020
Merged
8 tasks
tejasbubane added a commit to tejasbubane/rubocop that referenced this issue Sep 7, 2020
@tejasbubane
[Fix rubocop#8500] Add in? to AllowedMethods for `Lint/SafeNavigati…
1a1e8ef 
…onChain` cop

Closes rubocop#8500
bbatsov pushed a commit that referenced this issue Sep 8, 2020
@tejasbubane @bbatsov
[Fix #8500] Add in? to AllowedMethods for `Lint/SafeNavigationChain…
636d26a 
…` cop

Closes #8500
@johnnyshields
Copy link
Author

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Fix #8500] Add in? to AllowedMethods for Lint/SafeNavigationChain cop tejasbubane/rubocop
2 participants
@johnnyshields @marcandre