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

Allow users and rule authors to specify a reason per configured value #4611

Merged
merged 28 commits into from Jun 1, 2022
Merged

Allow users and rule authors to specify a reason per configured value #4611

merged 28 commits into from Jun 1, 2022

Commits on Jan 20, 2022

  1. implement first idea on values with reasons

    @marschwar
    marschwar committed Jan 20, 2022
    Copy the full SHA
    60023f3 View commit details
    Browse the repository at this point in the history

Commits on Jan 22, 2022

  1. Merge remote-tracking branch 'upstream/main' into feature/values-with… 

    …-reasons
    Markus Schwarz committed Jan 22, 2022
    Copy the full SHA
    83d9398 View commit details
    Browse the repository at this point in the history

Commits on Jan 23, 2022

  1. have ExplainedValues implement List by delegation

    Markus Schwarz committed Jan 23, 2022
    Copy the full SHA
    a736879 View commit details
    Browse the repository at this point in the history

  2. Merge remote-tracking branch 'upstream/main' into feature/values-with… 

    …-reasons
    Markus Schwarz committed Jan 23, 2022
    Copy the full SHA
    ea85090 View commit details
    Browse the repository at this point in the history

Commits on Feb 1, 2022

  1. Merge remote-tracking branch 'upstream/main' into feature/values-with… 

    …-reasons
    Markus Schwarz committed Feb 1, 2022
    Copy the full SHA
    fc68d64 View commit details
    Browse the repository at this point in the history

Commits on Feb 9, 2022

  1. Parse explained value defaults

    Markus Schwarz committed Feb 9, 2022
    Copy the full SHA
    3f14934 View commit details
    Browse the repository at this point in the history

  2. Merge remote-tracking branch 'upstream/main' into feature/values-with… 

    …-reasons
    Markus Schwarz committed Feb 9, 2022
    Copy the full SHA
    6cbb99f View commit details
    Browse the repository at this point in the history

  3. fix error message

    Markus Schwarz committed Feb 9, 2022
    Copy the full SHA
    1de89d6 View commit details
    Browse the repository at this point in the history

Commits on Feb 18, 2022

  1. Merge remote-tracking branch 'upstream/main' into feature/values-with… 

    …-reasons
    @marschwar
    marschwar committed Feb 18, 2022
    Copy the full SHA
    304e23a View commit details
    Browse the repository at this point in the history

  2. [WIP] support ExplainedValues in yaml

    @marschwar
    marschwar committed Feb 18, 2022
    Copy the full SHA
    0bbcdc6 View commit details
    Browse the repository at this point in the history

Commits on Mar 5, 2022

  1. Merge remote-tracking branch 'upstream/main' into feature/values-with… 

    …-reasons
    Markus Schwarz committed Mar 5, 2022
    Copy the full SHA
    0991416 View commit details
    Browse the repository at this point in the history

  2. print markdown value in DefaultValue directly

    Markus Schwarz committed Mar 5, 2022
    Copy the full SHA
    f19b593 View commit details
    Browse the repository at this point in the history

Commits on Mar 6, 2022

  1. quote values in yaml maps

    Markus Schwarz committed Mar 6, 2022
    Copy the full SHA
    5544c0d View commit details
    Browse the repository at this point in the history

Commits on Mar 7, 2022

  1. Update detekt-api/src/main/kotlin/io/gitlab/arturbosch/detekt/api/Con… 

    …figProperty.kt

Co-authored-by: Brais Gabín <braisgabin@gmail.com>
    @marschwar @BraisGabin
    marschwar and BraisGabin committed Mar 7, 2022
    Copy the full SHA
    87f1ea5 View commit details
    Browse the repository at this point in the history

  2. Update detekt-api/src/main/kotlin/io/gitlab/arturbosch/detekt/api/Con… 

    …figProperty.kt

Co-authored-by: Brais Gabín <braisgabin@gmail.com>
    @marschwar @BraisGabin
    marschwar and BraisGabin committed Mar 7, 2022
    Copy the full SHA
    d8010b6 View commit details
    Browse the repository at this point in the history

  3. add test case for missing reason

    Markus Schwarz committed Mar 7, 2022
    Copy the full SHA
    8b24d41 View commit details
    Browse the repository at this point in the history

  4. fail fast in case of invalid configuration

    Markus Schwarz committed Mar 7, 2022
    Copy the full SHA
    bd0261e View commit details
    Browse the repository at this point in the history

Commits on Mar 21, 2022

  1. Merge branch 'main' into feature/values-with-reasons

    Markus Schwarz committed Mar 21, 2022
    Copy the full SHA
    0156a4c View commit details
    Browse the repository at this point in the history

  2. rename explainedValues to valuesWithReason

    Markus Schwarz committed Mar 21, 2022
    Copy the full SHA
    204aaec View commit details
    Browse the repository at this point in the history

  3. add api documentation

    Markus Schwarz committed Mar 21, 2022
    Copy the full SHA
    c2eec92 View commit details
    Browse the repository at this point in the history

  4. resolve and suppress code smells

    Markus Schwarz committed Mar 21, 2022
    Copy the full SHA
    edcf8c3 View commit details
    Browse the repository at this point in the history

  5. suppress UNCHECKED_CAST

    Markus Schwarz committed Mar 21, 2022
    Copy the full SHA
    ecbce9f View commit details
    Browse the repository at this point in the history

Commits on Apr 21, 2022

  1. Merge remote-tracking branch 'upstream/main' into feature/values-with… 

    …-reasons
    Markus Schwarz committed Apr 21, 2022
    Copy the full SHA
    c0b1038 View commit details
    Browse the repository at this point in the history

Commits on May 22, 2022

  1. Merge remote-tracking branch 'upstream/main' into feature/values-with… 

    …-reasons

# Conflicts:
#	detekt-generator/src/test/kotlin/io/gitlab/arturbosch/detekt/generator/collection/ConfigurationSpec.kt
#	detekt-generator/src/test/kotlin/io/gitlab/arturbosch/detekt/generator/collection/RuleCollectorSpec.kt
    Markus Schwarz committed May 22, 2022
    Copy the full SHA
    fe99851 View commit details
    Browse the repository at this point in the history

Commits on May 24, 2022

  1. merge RuleCollectorSpec

    Markus Schwarz committed May 24, 2022
    Copy the full SHA
    8b4a23c View commit details
    Browse the repository at this point in the history

  2. reformat spec

    Markus Schwarz committed May 24, 2022
    Copy the full SHA
    23efa5a View commit details
    Browse the repository at this point in the history

Commits on May 29, 2022

  1. fix indentation of test code

    Markus Schwarz committed May 29, 2022
    Copy the full SHA
    4d2d279 View commit details
    Browse the repository at this point in the history

  2. Merge remote-tracking branch 'upstream/main' into feature/values-with… 

    …-reasons
    Markus Schwarz committed May 29, 2022
    Copy the full SHA
    b27a563 View commit details
    Browse the repository at this point in the history