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

chore: resolve new cops and some TODO cops #276

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

chore: resolve new cops and some TODO cops #276

wants to merge 7 commits into from

Commits on May 2, 2024

  1. chore: auto-correct some cops from TODO 

    - Layout/MultilineMethodCallBraceLayout
- Layout/MultilineMethodCallIndentation
- Lint/AssignmentInCondition (unsafe)
- Lint/ParenthesesAsGroupedExpression (unsafe)
    @doudou
    doudou committed May 2, 2024
    Configuration menu
    Copy the full SHA
    daf6710 View commit details
    Browse the repository at this point in the history

Commits on May 8, 2024

  1. chore: fix all useless assignment cops 

    This ... actually found probable bugs or inefficiencies
    @doudou
    doudou committed May 8, 2024
    Configuration menu
    Copy the full SHA
    4d61fce View commit details
    Browse the repository at this point in the history

  2. chore: enable Style/HashConversion

    @doudou
    doudou committed May 8, 2024
    Configuration menu
    Copy the full SHA
    2feab05 View commit details
    Browse the repository at this point in the history

  3. chore: enable some trivial new cops

    @doudou
    doudou committed May 8, 2024
    Configuration menu
    Copy the full SHA
    feb5353 View commit details
    Browse the repository at this point in the history

  4. fix: remove debug output

    @doudou
    doudou committed May 8, 2024
    Configuration menu
    Copy the full SHA
    7ae3b83 View commit details
    Browse the repository at this point in the history

  5. fix: require interface/core in roby itself

    @doudou
    doudou committed May 8, 2024
    Configuration menu
    Copy the full SHA
    a4e87ae View commit details
    Browse the repository at this point in the history

  6. fix: avoid using regexp when simple string tests are enough

    @doudou
    doudou committed May 8, 2024
    Configuration menu
    Copy the full SHA
    a0c2393 View commit details
    Browse the repository at this point in the history