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 #7077] Rename cops to meet the cop naming guideline #10725

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

[Fix #7077] Rename cops to meet the cop naming guideline #10725

wants to merge 7 commits into from

Commits on Jun 24, 2022

  1. Add a cop naming guideline

    @pirj
    pirj committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    9f4ad28 View commit details
    Browse the repository at this point in the history

  2. Rename Layout/IndentationConsistency to Layout/InconsistentIndentation

    @pirj
    pirj committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    7d3b837 View commit details
    Browse the repository at this point in the history

  3. Rename Lint/PercentStringArray to Lint/RedundantPercentWPunctuation

    @pirj
    pirj committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    be10b22 View commit details
    Browse the repository at this point in the history

  4. Rename Lint/PercentSymbolArray to Lint/RedundantPercentIPunctuation

    @pirj
    pirj committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    28f484a View commit details
    Browse the repository at this point in the history

  5. Rename Naming/VariableNumber to Naming/NumericVariableSuffix

    @pirj
    pirj committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    7ebd7dd View commit details
    Browse the repository at this point in the history

  6. Rename Style/DefWithParentheses to Style/RedundantMethodDefParentheses

    @pirj
    pirj committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    ba3cbf3 View commit details
    Browse the repository at this point in the history

  7. Rename Lint/DisjunctiveAssignmentInConstructor to Lint/OrAssignmentIn… 

    …Constructor

There is no consensus which term is appropriate, "or-assignment" or
"disjunctive assignment".
For consistency with `Lint/OrAssignmentToConstant`, and to save a bunch of bytes,
a shorter name was chosen.
    @pirj
    pirj committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    b409e45 View commit details
    Browse the repository at this point in the history