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

Separating ComplexMethod rule into CyclomaticComplexMethod and CognitiveComplexMethod #5442

Merged
merged 10 commits into from Oct 23, 2022
Merged

Separating ComplexMethod rule into CyclomaticComplexMethod and CognitiveComplexMethod #5442

merged 10 commits into from Oct 23, 2022

Commits on Oct 19, 2022

  1. feature: add rule CognitiveComplexMethod

    @sanggggg
    sanggggg committed Oct 19, 2022
    Copy the full SHA
    e5112cd View commit details
    Browse the repository at this point in the history

  2. test: add test for CognitiveComplexMethod rule

    @sanggggg
    sanggggg committed Oct 19, 2022
    Copy the full SHA
    d71cc8e View commit details
    Browse the repository at this point in the history

  3. move: ComplexMethod -> CyclomaticComplexMethod

    @sanggggg
    sanggggg committed Oct 19, 2022
    Copy the full SHA
    74749ba View commit details
    Browse the repository at this point in the history

  4. fix: remove referencing ComplexMethod

    @sanggggg
    sanggggg committed Oct 19, 2022
    Copy the full SHA
    812b119 View commit details
    Browse the repository at this point in the history

  5. fix: detekt correct

    @sanggggg
    sanggggg committed Oct 19, 2022
    Copy the full SHA
    7609ec4 View commit details
    Browse the repository at this point in the history

  6. fix: set alias

    @sanggggg
    sanggggg committed Oct 19, 2022
    Copy the full SHA
    cde1341 View commit details
    Browse the repository at this point in the history

  7. config: deactivate default cognitive complex method rule

    @sanggggg
    sanggggg committed Oct 19, 2022
    Copy the full SHA
    4545cd7 View commit details
    Browse the repository at this point in the history

  8. test: add positive and negative test cases

    @sanggggg
    sanggggg committed Oct 19, 2022
    Copy the full SHA
    b7bd1b0 View commit details
    Browse the repository at this point in the history

Commits on Oct 20, 2022

  1. Update detekt-rules-complexity/src/main/kotlin/io/gitlab/arturbosch/d… 

    …etekt/rules/complexity/CyclomaticComplexMethod.kt

Co-authored-by: Brais Gabín <braisgabin@gmail.com>
    @sanggggg @BraisGabin
    sanggggg and BraisGabin committed Oct 20, 2022
    Copy the full SHA
    236d3ce View commit details
    Browse the repository at this point in the history

  2. fix: remove conflicting activeByDefault and sync

    @sanggggg
    sanggggg committed Oct 20, 2022
    Copy the full SHA
    2a0ae60 View commit details
    Browse the repository at this point in the history