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.

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

Disable maven exclusion semantics in Spring Dependency Management Plugin #940

Merged

Conversation

candrews
Copy link
Owner

Starting with Spring Dependency Management Plugin version 1.1.0, invalid POMs cause the plugin to fail. Unfortunately, Sonar has dependencies that have invalid POMs, and it doesn't appear that Sonar is going to change to use dependencies with valid POMs any time soon.

Therefore, in order to continuing using the name.remal.sonarlint plugin, there are only two options:

  1. Downgrade Sonar to a version that has dependencies all with valid POMs. That could be done by adding a dependency on: sonarlintCorePlugins("org.sonarsource.java:sonar-java-plugin:7.18.0.31443") However, it's not ideal to be stuck on an old version for an indeterminate amount of time.
  2. Disable Spring Dependency Management Plugin's new maven exclusion semantics. There doesn't appear to be a downside for this project in doing so.

See: spring-gradle-plugins/dependency-management-plugin#365
See: https://community.sonarsource.com/t/sonar-java-plugin-has-dependencies-with-invalid-poms/96334/6

@candrews candrews enabled auto-merge (rebase) July 27, 2023 15:59
@candrews candrews force-pushed the disable-spring-dependency-management-maven-exclusion-semantics branch from 332845c to 6f1e832 Compare July 27, 2023 16:11
Starting with Spring Dependency Management Plugin version 1.1.0, invalid
POMs cause the plugin to fail. Unfortunately, Sonar has dependencies
that have invalid POMs, and it doesn't appear that Sonar is going to
change to use dependencies with valid POMs any time soon.

Therefore, in order to continuing using the name.remal.sonarlint plugin,
there are only two options:
1. Downgrade Sonar to a version that has dependencies all with valid
  POMs. That could be done by adding a dependency on:
  `sonarlintCorePlugins("org.sonarsource.java:sonar-java-plugin:7.18.0.31443")`
  However, it's not ideal to be stuck on an old version for an
  indeterminate amount of time.
2. Disable Spring Dependency Management Plugin's new maven exclusion
   semantics. There doesn't appear to be a downside for this project in
   doing so.

See: spring-gradle-plugins/dependency-management-plugin#365
See: https://community.sonarsource.com/t/sonar-java-plugin-has-dependencies-with-invalid-poms/96334/6
@candrews candrews force-pushed the disable-spring-dependency-management-maven-exclusion-semantics branch from 6f1e832 to 4ab1069 Compare July 27, 2023 16:12
@candrews candrews merged commit 4e21cef into master Jul 27, 2023
@candrews candrews deleted the disable-spring-dependency-management-maven-exclusion-semantics branch July 27, 2023 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant