Skip to content

BUILD-11571: Pin check-sca version with better vault-error-output#303

Merged
bwalsh434 merged 1 commit into
masterfrom
feat/bwalsh/BUILD-11571-pin-check-sca-version-with-better-vault-error-output
Jun 15, 2026
Merged

BUILD-11571: Pin check-sca version with better vault-error-output#303
bwalsh434 merged 1 commit into
masterfrom
feat/bwalsh/BUILD-11571-pin-check-sca-version-with-better-vault-error-output

Conversation

@bwalsh434

@bwalsh434 bwalsh434 commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Follow up to #300 so that this newly committed to master version of check-sca gets used in this repo as well as all of the other repos via the org ruleset.

Test Plan

@bwalsh434 bwalsh434 marked this pull request as ready for review June 15, 2026 16:54
@bwalsh434 bwalsh434 requested a review from a team as a code owner June 15, 2026 16:54
Copilot AI review requested due to automatic review settings June 15, 2026 16:54
@hashicorp-vault-sonar-prod

hashicorp-vault-sonar-prod Bot commented Jun 15, 2026

Copy link
Copy Markdown

BUILD-11571

@gitar-bot

gitar-bot Bot commented Jun 15, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

Pins the check-sca action version to incorporate improved Vault error messaging. No issues found.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@sonarqubecloud

Copy link
Copy Markdown

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the repository’s SCA verification workflow to use the newly committed check-sca action revision (follow-up to PR #300), ensuring improved Vault-auth failure output is picked up here and via the org ruleset.

Changes:

  • Bump the pinned SonarSource/ci-github-actions/check-sca action SHA to a newer commit on master.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/check-sca.yml
@bwalsh434 bwalsh434 merged commit d17388b into master Jun 15, 2026
23 checks passed
@bwalsh434 bwalsh434 deleted the feat/bwalsh/BUILD-11571-pin-check-sca-version-with-better-vault-error-output branch June 15, 2026 17:26
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.

3 participants