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

Enable Local Build Caching and Apply Optimizations #2656

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

tylerbertrand
Copy link
Contributor

Enable local build caching with Develocity, so that goals whose inputs have not changed do not need to be re-executed. This should help reduce local build times when making contributions to Feign (note that it is required to run the clean lifecycle phase - i.e. ./mvnw clean install vs just ./mvnw install - to store outputs to the build cache).

Resolve a cache miss by normalizing manifest files so that timestamps in them do not cause goals to be needlessly re-executed. This cache miss was discovered by executing the Develocity Build Validation Scripts for Maven against the project.

Apply the develocity-actions/setup-maven action to help surface Build Scan links

This is an identifier captured by Develocity used to uniquely represent a given workspace should not be committed
Don't add PR comment with build scan links - scans aren't published for PR builds against forks
@tylerbertrand tylerbertrand force-pushed the tylerbertrand/develocity-optimizations branch from f2d303f to 7a4c6b1 Compare November 14, 2024 18:36
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Some suggestions could not be made:

  • annotation-error-decoder/pom.xml
    • lines 49-48
  • apt-test-generator/pom.xml
    • lines 62-61
  • benchmark/pom.xml
    • lines 70-69
  • core/pom.xml
    • lines 37-36
  • googlehttpclient/pom.xml
    • lines 54-53
  • hc5/pom.xml
    • lines 67-66
  • httpclient/pom.xml
    • lines 75-74
  • hystrix/pom.xml
    • lines 78-77
  • jackson-jaxb/pom.xml
    • lines 82-83
  • java11/pom.xml
    • lines 50-49
  • jaxb-jakarta/pom.xml
    • lines 34-35
    • lines 55-57
  • jaxb/pom.xml
    • lines 45-49
  • jaxrs2/pom.xml
    • lines 103-102
  • jaxrs3/pom.xml
    • lines 89-88
  • jaxrs4/pom.xml
    • lines 83-82
  • kotlin/pom.xml
    • lines 71-70
  • okhttp/pom.xml
    • lines 59-58
  • pom.xml
    • lines 594-595
  • reactive/pom.xml
    • lines 98-97
  • ribbon/pom.xml
    • lines 83-82
  • soap-jakarta/pom.xml
    • lines 34-35
    • lines 63-62
    • lines 79-81
  • soap/pom.xml
    • lines 55-60
    • lines 70-71

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