Skip to content

BUILD-9332 disable-cache and cache-paths options for Maven and Gradle, update docs#119

Merged
mikolaj-matuszny-ext-sonarsource merged 1 commit intomasterfrom
feat/mmatuszny/BUILD-9332-caching-maven
Oct 10, 2025
Merged

BUILD-9332 disable-cache and cache-paths options for Maven and Gradle, update docs#119
mikolaj-matuszny-ext-sonarsource merged 1 commit intomasterfrom
feat/mmatuszny/BUILD-9332-caching-maven

Conversation

@mikolaj-matuszny-ext-sonarsource
Copy link
Copy Markdown
Contributor

@mikolaj-matuszny-ext-sonarsource mikolaj-matuszny-ext-sonarsource commented Oct 7, 2025 •
edited
Loading

BUILD-9332
Adds option to disable cache and customize cache paths for Maven
Adjust cache-paths for Gradle to have greater customizability

@mikolaj-matuszny-ext-sonarsource mikolaj-matuszny-ext-sonarsource force-pushed the feat/mmatuszny/BUILD-9332-caching-maven branch from 0877b42 to d0a7e8a Compare October 7, 2025 10:17
@mikolaj-matuszny-ext-sonarsource mikolaj-matuszny-ext-sonarsource changed the title BUILD-9332 disable-cache and cache-paths options for Maven BUILD-9332 disable-cache and cache-paths options for Maven and update docs Oct 7, 2025
@mikolaj-matuszny-ext-sonarsource
Copy link
Copy Markdown
Contributor Author

mikolaj-matuszny-ext-sonarsource commented Oct 7, 2025

Testing looks good:
https://github.com/SonarSource/sonar-dummy-maven-enterprise/actions/runs/18309668908/job/52135089785?pr=98
https://github.com/SonarSource/sonar-dummy-maven-enterprise/actions/runs/18309566406/job/52134772895?pr=98

@mikolaj-matuszny-ext-sonarsource mikolaj-matuszny-ext-sonarsource force-pushed the feat/mmatuszny/BUILD-9332-caching-maven branch 3 times, most recently from ad7c565 to 2504daa Compare October 7, 2025 11:11
@mikolaj-matuszny-ext-sonarsource
Copy link
Copy Markdown
Contributor Author

mikolaj-matuszny-ext-sonarsource commented Oct 7, 2025

Gradle testing:

@mikolaj-matuszny-ext-sonarsource mikolaj-matuszny-ext-sonarsource changed the title BUILD-9332 disable-cache and cache-paths options for Maven and update docs BUILD-9332 disable-cache and cache-paths options for Maven and Gradle, update docs Oct 7, 2025
julien-carsique-sonarsource
julien-carsique-sonarsource requested changes Oct 9, 2025
View reviewed changes
Comment thread build-maven/action.yml Outdated
Comment thread build-maven/action.yml
Comment thread config-maven/action.yml
Comment thread config-maven/action.yml Outdated
Comment thread config-maven/action.yml Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
@mikolaj-matuszny-ext-sonarsource mikolaj-matuszny-ext-sonarsource force-pushed the feat/mmatuszny/BUILD-9332-caching-maven branch 4 times, most recently from 89e444d to 9a31121 Compare October 9, 2025 11:11
@mikolaj-matuszny-ext-sonarsource
Copy link
Copy Markdown
Contributor Author

mikolaj-matuszny-ext-sonarsource commented Oct 9, 2025

Maven testing looks ok - https://github.com/SonarSource/sonar-dummy-maven-enterprise/actions/runs/18374493325/job/52345203867?pr=100#step:5:452

@mikolaj-matuszny-ext-sonarsource mikolaj-matuszny-ext-sonarsource force-pushed the feat/mmatuszny/BUILD-9332-caching-maven branch from 9a31121 to 3c7510a Compare October 9, 2025 11:20
julien-carsique-sonarsource
julien-carsique-sonarsource approved these changes Oct 9, 2025
View reviewed changes
Comment thread README.md Outdated
@mikolaj-matuszny-ext-sonarsource mikolaj-matuszny-ext-sonarsource force-pushed the feat/mmatuszny/BUILD-9332-caching-maven branch 2 times, most recently from 43fa901 to 7a278ab Compare October 10, 2025 07:09
@mikolaj-matuszny-ext-sonarsource mikolaj-matuszny-ext-sonarsource force-pushed the feat/mmatuszny/BUILD-9332-caching-maven branch from 7a278ab to 1948d58 Compare October 10, 2025 07:09
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Oct 10, 2025

🤖 Pull Request summary

Adds customizable caching configuration for Maven and Gradle build actions.

• New caching options: Added cache-paths and disable-caching inputs to both Maven and Gradle actions
• Default cache paths: Updated Gradle action to explicitly define default cache directories (~/.gradle/caches, ~/.gradle/wrapper)
• Documentation updates: Added caching configuration sections with usage examples for both build types
• Vault permissions clarification: Updated documentation to specify SonarQube credentials are only fetched when needed
• Minor text corrections: Fixed various documentation formatting and wording inconsistencies

Review focus: Verify the cache path handling logic, especially how empty cache-paths input is handled in the Maven action and whether the default behavior is preserved correctly.

💬 Please send your feedback

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues
0 Dependency risks

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Oct 10, 2025 •
edited
Loading

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues
0 Dependency risks

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@mikolaj-matuszny-ext-sonarsource mikolaj-matuszny-ext-sonarsource merged commit c1a30df into master Oct 10, 2025
11 checks passed
@mikolaj-matuszny-ext-sonarsource mikolaj-matuszny-ext-sonarsource deleted the feat/mmatuszny/BUILD-9332-caching-maven branch October 10, 2025 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@julien-carsique-sonarsource julien-carsique-sonarsource julien-carsique-sonarsource approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mikolaj-matuszny-ext-sonarsource @julien-carsique-sonarsource