Update push-trigger.yml#113
Conversation
Signed-off-by: Rajapandi M <138785181+rajapandi1234@users.noreply.github.com>
WalkthroughUpdated a GitHub Actions workflow file to reference the gradlew-sonar-analysis workflow by specific commit hash instead of a branch name. This change pins the workflow version to a particular commit rather than tracking branch updates. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.github/workflows/push-trigger.yml:
- Line 34: The workflow currently pins the reusable workflow using a short ref
"uses: mosip/kattu/.github/workflows/gradlew-sonar-analysis.yml@dsd9685";
replace that suffix with the full 40-character commit SHA of the exact commit
you want to pin (e.g., ...gradlew-sonar-analysis.yml@<40-char-sha>) so the
reusable workflow is immutably pinned; locate the uses: line in the
push-trigger.yml and update it to reference the full commit SHA (you can obtain
the SHA from the upstream repo via git ls-remote or the commit history on
GitHub).
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 38201e72-039d-49a0-a099-a8abba0d77e6
📒 Files selected for processing (1)
.github/workflows/push-trigger.yml
Summary by CodeRabbit