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

Publish build scans to develocity.apache.org #7891

Merged
merged 1 commit into from
Jan 20, 2025

Conversation

clayburn
Copy link
Contributor

This PR migrates the Geode project to publish Build Scans to the the new Develocity instance at develocity.apache.org.

Additionally, this PR migrates from the legacy Gradle Enterprise plugin to the renamed Develocity plugin and sets a projectId for use by Develocity.

https://issues.apache.org/jira/browse/GEODE-10457

For all changes:

  • Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

  • Has your PR been rebased against the latest commit within the target branch (typically develop)?

  • Is your initial contribution a single, squashed commit?

  • Does gradlew build run cleanly?

  • Have you written or updated unit tests to verify your changes?

  • If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Copy link
Member

@CalvinKirs CalvinKirs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but I’m not an expert with Gradle, and the build logs also show that our Gradle version is outdated. Would you be willing to help us upgrade to the latest version?

@clayburn
Copy link
Contributor Author

@CalvinKirs - upgrading the Gradle version is likely something I won't have capacity for soon.

@CalvinKirs
Copy link
Member

Thank you for your contribution:)

@CalvinKirs CalvinKirs merged commit 6a51368 into apache:develop Jan 20, 2025
14 of 17 checks passed
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.

2 participants