Skip to content

Disable Style/NumberedParametersLimit permanently

Description of the proposal

Because Style/NumberedParameters was enabled and configured to disallow any use of numbered parameters (_1 etc.) in !153 (merged) there's no need to enable Style/NumberedParametersLimit to save some CPU cycles.

This MR has been repurposed.

See old description

As discussed in !153 (comment 1237677111)

Regardless of Style/NumberedParameters cop enabled or not, we should limit this to 1.

In gitlab-org/gitlab, 0 offenses currently

29491 files inspected, no offenses detected

Check-list

  • Mention this proposal in the relevant Slack channels (e.g. #development, #backend, #frontend)
  • [-] If there is a choice to make between two potential styles, set up an emoji vote in the MR:
    • CHOICE_A: 🅰
    • CHOICE_B: 🅱
    • Vote yourself for both choices so that people know these are the choices
  • The MR doesn't have significant objections, and is getting a majority of 👍 vs 👎 (remember that we don't need to reach a consensus)
  • [-] (If applicable) One style is getting a majority of vote (compared to the other choice)
  • [-] (If applicable) Update the MR with the chosen style
  • Follow the review process as usual

/cc @gitlab-org/maintainers/rails-backend

Edited by Peter Leitzen

Merge request reports

Loading