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

Use Develocity Maven extension #119

Merged
merged 13 commits into from
Mar 20, 2024

Conversation

snoopcheri
Copy link
Member

@snoopcheri snoopcheri commented Mar 14, 2024

This PR

  • updates the embedded version of the Maven extension to 1.21-rc-3
  • configures system properties for both the old and new Maven extension

Before this PR can be merged:

  • update embedded Maven extension to 1.21 once it is released (and remove Gradle's public repository as well)
  • check that integration tests are all green, as soon as scans.gradle.org is updated to 2024.1 (at the moment, that instance doesn't accept the newest Maven extension version)

Signed-off-by: Daniel Clausen <dclausen@gradle.com>
Only if neither the deprecated nor the new version of Maven extension
is found, we want to add the embedded Maven extension to the class path.

Signed-off-by: Daniel Clausen <dclausen@gradle.com>
Since this type is used for parameters of public methods, the visibility
scope should be adjusted.

Signed-off-by: Daniel Clausen <dclausen@gradle.com>
Older Maven extensions will simply ignore the new system properties.
Newer Maven extensions will use the new system properties and therefore
build will not produce deprecation warnings.

Signed-off-by: Daniel Clausen <dclausen@gradle.com>
Signed-off-by: Daniel Clausen <dclausen@gradle.com>
This has been adjusted, because Develocity use terms-of-use
instead of terms-of-service.

Signed-off-by: Daniel Clausen <dclausen@gradle.com>
Signed-off-by: Daniel Clausen <dclausen@gradle.com>
…class

Signed-off-by: Daniel Clausen <dclausen@gradle.com>
* the line for the v2.0.0 of the Bamboo plugin must remain unchanged
* the line for "Next version" is the version released when DV 2024.1
  and Maven extension 1.21 are released

Signed-off-by: Daniel Clausen <dclausen@gradle.com>
Signed-off-by: Daniel Clausen <dclausen@gradle.com>
Signed-off-by: Daniel Clausen <dclausen@gradle.com>
@snoopcheri snoopcheri force-pushed the snoopcheri/33446-configure-develocity-plugin branch from 8b776fa to c435fe0 Compare March 19, 2024 12:20
scans.gradle.org unfortunately does not accept RC versions of build agents.
This change detects this case and makes sure that the test buildScanIsPublished
still passes.

Signed-off-by: Daniel Clausen <dclausen@gradle.com>
As soon as the Maven extension is released, we have to
* update the embedded Maven extension version to a non-RC one
* remove the Gradle-internal repositories
* remove the hack in MavenInjectionTest which detects RC versions

Signed-off-by: Daniel Clausen <dclausen@gradle.com>
@snoopcheri snoopcheri added the enhancement New feature or request label Mar 20, 2024
@snoopcheri snoopcheri marked this pull request as ready for review March 20, 2024 13:05
@snoopcheri snoopcheri merged commit 7771540 into main Mar 20, 2024
10 checks passed
@snoopcheri snoopcheri deleted the snoopcheri/33446-configure-develocity-plugin branch March 20, 2024 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants