-
Notifications
You must be signed in to change notification settings - Fork 33
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
Add Convention Develocity Maven extension #636
base: main
Are you sure you want to change the base?
Conversation
2fed8d0
to
ddd20ea
Compare
gradle-enterprise-conventions-maven-extension/extension/src/main/java/com/example/CiUtils.java
Outdated
Show resolved
Hide resolved
gradle-enterprise-conventions-maven-extension/extension/pom.xml
Outdated
Show resolved
Hide resolved
gradle-enterprise-conventions-maven-extension/extension/README.md
Outdated
Show resolved
Hide resolved
2e47869
to
591f7a7
Compare
gradle-enterprise-conventions-maven-extension/example-builds/maven_3.6.x/.mvn/extensions.xml
Outdated
Show resolved
Hide resolved
gradle-enterprise-conventions-maven-extension/extension/.mvn/extensions.xml
Outdated
Show resolved
Hide resolved
gradle-enterprise-conventions-maven-extension/extension/.mvn/gradle-enterprise.xml
Outdated
Show resolved
Hide resolved
gradle-enterprise-conventions-maven-extension/extension/src/main/java/com/example/CiUtils.java
Outdated
Show resolved
Hide resolved
...ntions-maven-extension/extension/src/main/java/com/example/CustomGradleEnterpriseConfig.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i agree with what Daz said about trying to dogfood this extension.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor comments
gradle-enterprise-conventions-maven-extension/example-builds/maven_3.6.x/pom.xml
Outdated
Show resolved
Hide resolved
gradle-enterprise-conventions-maven-extension/example-builds/maven_3.6.x/pom.xml
Outdated
Show resolved
Hide resolved
gradle-enterprise-conventions-maven-extension/example-builds/maven_3.9.x/pom.xml
Outdated
Show resolved
Hide resolved
gradle-enterprise-conventions-maven-extension/extension/.mvn/extensions.xml
Outdated
Show resolved
Hide resolved
gradle-enterprise-conventions-maven-extension/extension/.mvn/gradle-enterprise.xml
Outdated
Show resolved
Hide resolved
...conventions-maven-extension/example-builds/maven_3.6.x/.mvn/wrapper/maven-wrapper.properties
Outdated
Show resolved
Hide resolved
@clayburn I have not gotten to review this, yet, but please make sure all of this runs on Java 8, incl. the build itself. Similarly, the approach of this convention should work on Maven 3.6, 3.7, 3.8, and 3.9. thanks. |
4329f7f
to
d97f488
Compare
...le-enterprise-conventions-maven-extension/extension/src/main/resources/gradle-enterprise.xml
Outdated
Show resolved
Hide resolved
.github/workflows/gradle-enterprise-conventions-maven-extension-verification.yml
Outdated
Show resolved
Hide resolved
f227520
to
573db6b
Compare
...sion/extension/src/main/java/com/myorg/ConventionGradleEnterpriseMavenExtensionListener.java
Outdated
Show resolved
Hide resolved
This needs some attention. I'm going to close it but leave the branch around for now until we want to prioritize it again. |
9a1dace
to
75f60b7
Compare
convention-develocity-maven-extension/example/.mvn/extensions.xml
Outdated
Show resolved
Hide resolved
1e7ce45
to
f4cc495
Compare
f4cc495
to
19e0948
Compare
This PR adds a sample Maven extension that applies Develocity, Common Custom User Data Maven extension, and a reasonable set of defaults for configuring build scans and build caching. The goal is to provide a sample that makes it simple for organizations to create and host their own, customized version of this extension in order to more easily and consistently apply Develocity configurations across an organization.