-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
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
rubocop: Further trim Naming/MethodParameterName
allowlist
#14944
rubocop: Further trim Naming/MethodParameterName
allowlist
#14944
Conversation
- This either stands for "file" but more often than not "formula".
- I couldn't figure out what this would stand for, so I asked on Slack if anyone had better ideas otherwise I'd go with "ostrich" or "octopus". Rylan suggested "option", which is very sensible.
Review period will end on 2023-03-14 at 00:00:00 UTC. |
Review period skipped due to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good so far!
"f", | ||
"o", | ||
"pr", | ||
"pr", # TODO: Remove if https://github.com/rubocop/rubocop/pull/11690 is merged or we change the variable names. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might want to move f to another PR so the tap syntax changes don't block this one?
- Part of Homebrew/brew#14944. - This stands for "formula".
- Part of Homebrew/brew#14944. - This stands for formula.
brew style
with your changes locally?brew typecheck
with your changes locally?brew tests
with your changes locally?brew typecheck
which caught some occurrences I hadn't picked up, I ranbrew tests
which caught some more, and I ranbrew style
which caught even more. Worth a second set of eyes though?