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

Add rule call constructor #1208

Merged
merged 9 commits into from Apr 17, 2022
Merged

Add rule call constructor #1208

merged 9 commits into from Apr 17, 2022

Commits on Apr 17, 2022

  1. Add Rule to check method call constructor

    @muno92 @ondrejmirtes
    muno92 authored and ondrejmirtes committed Apr 17, 2022
    Copy the full SHA
    f0579f0 View commit details
    Browse the repository at this point in the history

  2. Register the rules on level 2 so that they're executed (but only with… 

    … bleedingEdge)
    @ondrejmirtes
    ondrejmirtes committed Apr 17, 2022
    Copy the full SHA
    2ceb04c View commit details
    Browse the repository at this point in the history

  3. Use RuleErrorBuilder

    @ondrejmirtes
    ondrejmirtes committed Apr 17, 2022
    Copy the full SHA
    1458d22 View commit details
    Browse the repository at this point in the history

  4. Improve error messages

    @ondrejmirtes
    ondrejmirtes committed Apr 17, 2022
    Copy the full SHA
    17b0800 View commit details
    Browse the repository at this point in the history

  5. Be case-insensitive

    @ondrejmirtes
    ondrejmirtes committed Apr 17, 2022
    Copy the full SHA
    71a6640 View commit details
    Browse the repository at this point in the history

  6. Adjust test

    @ondrejmirtes
    ondrejmirtes committed Apr 17, 2022
    Copy the full SHA
    72a3a3d View commit details
    Browse the repository at this point in the history

  7. Behave the same way with "self" like with the actual class name

    @ondrejmirtes
    ondrejmirtes committed Apr 17, 2022
    Copy the full SHA
    4daa27c View commit details
    Browse the repository at this point in the history

  8. Test files must have a namespace

    @ondrejmirtes
    ondrejmirtes committed Apr 17, 2022
    Copy the full SHA
    f7cd97a View commit details
    Browse the repository at this point in the history

  9. All of these should be a violation

    @ondrejmirtes
    ondrejmirtes committed Apr 17, 2022
    Copy the full SHA
    18930cc View commit details
    Browse the repository at this point in the history