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.

Sign up for GitHub

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

Fix Style/EachForSimpleLoop handling for blocks with a parameter #11681

Closed
wants to merge 1 commit into from
Closed

Fix Style/EachForSimpleLoop handling for blocks with a parameter #11681

wants to merge 1 commit into from

Commits on Mar 8, 2023

  1. Style/EachForSimpleLoop: Do not warn if block param is used 

    The simple loop cop documents that "This check only applies if the
block takes no parameters", however that is not how it behaves.

This commit updates the cop's behavior to only apply if the block
does not have any parameters.
    @j1wilmot
    j1wilmot committed Mar 8, 2023
    Configuration menu
    Copy the full SHA
    ffd5330 View commit details
    Browse the repository at this point in the history