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

Drop deprecated RulesDefinitionXmlLoader #550

Merged
merged 6 commits into from
May 26, 2022
Merged

Drop deprecated RulesDefinitionXmlLoader #550

merged 6 commits into from
May 26, 2022

Commits on May 14, 2022

  1. deps: drop deprecated RulesDefinitionXmlLoader 

    - RulesDefinitionXmlLoader is deprecated and will be dropped in SQ 10
- RulesDefinitionXmlLoader does not populate the OWASP categories and
can't be extended (see spotbugs#392)
- Loading/unloading the plugins dynamically from jar files leads to file
leaks because SpotBugs does not close the classloaders (solves #128)
- The Groovy scripts seem to need some fixes to work with Groovy 4
    @gtoison
    gtoison committed May 14, 2022
    Configuration menu
    Copy the full SHA
    20eeed3 View commit details
    Browse the repository at this point in the history

  2. deps: removed usage of dependencies no longer provided 

    The sonar-java-plugin is provided by the SonarQube server but not its
transient dependencies
- Removed usage of Guava
- Replaced commons-lang usage by commons-lang3
    @gtoison
    gtoison committed May 14, 2022
    Configuration menu
    Copy the full SHA
    fb08eac View commit details
    Browse the repository at this point in the history

  3. fix:issues detected by SonarQube analysis

    @gtoison
    gtoison committed May 14, 2022
    Configuration menu
    Copy the full SHA
    0376683 View commit details
    Browse the repository at this point in the history

Commits on May 21, 2022

  1. fix: issue detected by SQ analysis

    @gtoison
    gtoison committed May 21, 2022
    Configuration menu
    Copy the full SHA
    5c61c77 View commit details
    Browse the repository at this point in the history

  2. deps: removed the exclude, just make sure we wont use it

    @gtoison
    gtoison committed May 21, 2022
    Configuration menu
    Copy the full SHA
    199b7fa View commit details
    Browse the repository at this point in the history

  3. fix: removed unused code and test resources

    @gtoison
    gtoison committed May 21, 2022
    Configuration menu
    Copy the full SHA
    47ce39d View commit details
    Browse the repository at this point in the history