Skip to content

CLP-1075: Bump ci-github-actions references from v1 to v2 - #546

Merged
guillaume-dequenne merged 2 commits into
masterfrom
chore/mgeorgiou/CLP-1075-bumpCiActionsV2
Oct 1, 2026
Merged

guillaume-dequenne merged 2 commits into
masterfrom
chore/mgeorgiou/CLP-1075-bumpCiActionsV2

Conversation

@mary-georgiou

Copy link
Copy Markdown
Contributor

CLP-1075

Bumps all SonarSource/ci-github-actions/*@v1 references to @v2:

  • .github/workflows/pr-cleanup.yml:11 — pr_cleanup@v1 -> @v2
  • .github/workflows/build.yml:46 — build-maven@v1 -> @v2
  • .github/workflows/build.yml:92 — code-signing@v1 -> @v2
  • .github/workflows/build.yml:284 — promote@v1 -> @v2
  • .github/workflows/build.yml:354 — promote@v1 -> @v2

🤖 Generated with Claude Code

Bump SonarSource/ci-github-actions/*@v1 to @v2 (pr_cleanup, build-maven, code-signing, promote).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@hashicorp-vault-sonar-prod

hashicorp-vault-sonar-prod Bot commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

CLP-1075

@datadog-sonarsource

This comment has been minimized.

@sonarqube-next

sonarqube-next Bot commented Oct 1, 2026

Copy link
Copy Markdown

@mary-georgiou
mary-georgiou marked this pull request as ready for review October 1, 2026 11:25

@guillaume-dequenne guillaume-dequenne left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@guillaume-dequenne
guillaume-dequenne merged commit 39a3c79 into master Oct 1, 2026
10 checks passed
@guillaume-dequenne
guillaume-dequenne deleted the chore/mgeorgiou/CLP-1075-bumpCiActionsV2 branch October 1, 2026 12:21
@gitar-bot

gitar-bot Bot commented Oct 13, 2026

Copy link
Copy Markdown
Code Review 👍 Approved with suggestions 0 closed / 1 findings

🟡 Medium risk · CI workflows switch build, signing, and promotion actions to v2; their implementations are unseen.

Bumps all SonarSource/ci-github-actions/* references from @v1 to @v2 across five workflow steps. A comment at .github/workflows/build.yml:264 still references promote@v1 and should be updated to @v2 for consistency.

💡 Quality: Comment still says promote@v1 after the bump to v2

📄 .github/workflows/build.yml:264

This PR moves every SonarSource/ci-github-actions/* reference to @v2, including the promote-npm job at line 354. The comment in the npm deploy step (line 264) still says the job uses ci-github-actions/promote@v1. The workflow runs the same either way, but anyone reading that comment to understand the buildInfo.env contract (ARTIFACTORY_DEPLOY_REPO/PROJECT_VERSION) is pointed at the wrong major version. Update the comment to say @v2.

Change the comment to match the new action version.
# The promote-npm job (ci-github-actions/promote@v2) reads these from the build-info to derive the
🤖 Prompt for agents
Code Review: Bumps all `SonarSource/ci-github-actions/*` references from `@v1` to `@v2` across five workflow steps. A comment at `.github/workflows/build.yml:264` still references `promote@v1` and should be updated to `@v2` for consistency.

1. 💡 Quality: Comment still says promote@v1 after the bump to v2
   Files: .github/workflows/build.yml:264

   This PR moves every `SonarSource/ci-github-actions/*` reference to `@v2`, including the `promote-npm` job at line 354. The comment in the npm deploy step (line 264) still says the job uses `ci-github-actions/promote@v1`. The workflow runs the same either way, but anyone reading that comment to understand the buildInfo.env contract (ARTIFACTORY_DEPLOY_REPO/PROJECT_VERSION) is pointed at the wrong major version. Update the comment to say `@v2`.

   Fix (Change the comment to match the new action version.):
   # The promote-npm job (ci-github-actions/promote@v2) reads these from the build-info to derive the

Verification

Functional validation No results

Rules No rules evaluated

Auto-approval Not enabled

Set up

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Counting what did not apply, without listing it.

Comment with these commands to change the behavior for this request:

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

Was this helpful? React with 👍 / 👎 | Gitar

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