Skip to content

BUILD-8957 use local action checkout#74

Merged
julien-carsique-sonarsource merged 1 commit intomasterfrom
fix/jcarsique/BUILD-8957-selfRef
Sep 2, 2025
Merged

BUILD-8957 use local action checkout#74
julien-carsique-sonarsource merged 1 commit intomasterfrom
fix/jcarsique/BUILD-8957-selfRef

Conversation

@julien-carsique-sonarsource
Copy link
Copy Markdown
Contributor

@julien-carsique-sonarsource julien-carsique-sonarsource commented Sep 2, 2025

BUILD-8957

Refer to the local checkout to use other actions from the same repository, instead of a hardcoded reference to master.
There is no available environment variable to refer the proper hash, while the code has been checkouted with the proper hash.
A symlink is used because GitHub Actions rejects .. relative path in the uses: field.

@julien-carsique-sonarsource julien-carsique-sonarsource requested a review from a team September 2, 2025 09:37
@julien-carsique-sonarsource julien-carsique-sonarsource force-pushed the fix/jcarsique/BUILD-8957-selfRef branch from f62827f to c357267 Compare September 2, 2025 13:53
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Sep 2, 2025

🤖 Pull Request summary

Refactors GitHub Actions to use local symlinks instead of remote references.

Action dependencies: Replaced remote SonarSource/ci-github-actions/*@master calls with local symlinked actions in .actions/ directory
Build consolidation: Combined separate build tool setup steps into the main parameter setting step for cleaner workflow structure
Cache key standardization: Updated cache key formats to use consistent cache-${{ runner.os }}- prefix pattern
Restore keys simplification: Removed multiline restore-keys syntax in favor of single-line format

Review focus: Verify that the symlink creation logic correctly points to existing action directories and that the new local action references work across all build environments.

💬 Please send your feedback

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

@julien-carsique-sonarsource julien-carsique-sonarsource merged commit c891415 into master Sep 2, 2025
10 checks passed
@julien-carsique-sonarsource julien-carsique-sonarsource deleted the fix/jcarsique/BUILD-8957-selfRef branch September 2, 2025 14:02
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