Skip to content
This repository has been archived by the owner on May 4, 2023. It is now read-only.

Add support for ignore property in codiga.yml #239

Merged
merged 11 commits into from
Jan 19, 2023
Merged

Conversation

picimako
Copy link
Contributor

Changes

  • CodigaConfigFileUtil
    • Replaced the PSI based file/configuration parsing with Jackson-based deserialization, so that extending the parsing for future functionalities becomes easier.
      • Related model classes are: CodigaYmlConfig, RulesetIgnore, RuleIgnore
  • RosieRulesCache
    • Replaced the cached ruleset names with the deserialized CodigaYmlConfig.
      • Also updated its usages in RosieRulesCacheUpdateHandler.
    • Renamed getRosieRulesForLanguage() to getRosieRules(), and added logic to filter the rules based on the ignore configuration and passed in path of the currently analyzed file.
  • codiga-config.yaml
    • Updated the schema file to include the ignore property.
  • CodigaRulesetContentInspection
    • Updated it to validate only items in the rulesets property, and exclude string literals under prefix .
  • CodigaRulesetReferenceContributor
    • Updated it to provide links to only items in the rulesets property, and exclude string literals under prefix.

@dastrong-codiga dastrong-codiga merged commit bde456e into main Jan 19, 2023
@dastrong-codiga dastrong-codiga deleted the codiga-yml-ignore branch January 19, 2023 16:14
@dastrong-codiga dastrong-codiga restored the codiga-yml-ignore branch January 19, 2023 16:14
@dastrong-codiga dastrong-codiga deleted the codiga-yml-ignore branch January 19, 2023 16:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants