Skip to content

BUILD-11815: Remove SIGNING_PROFILE workaround from release pipeline#473

Merged
okorach-sonar merged 1 commit into
mainfrom
chore/mmolnar/BUILD-11815-removeSigningProfileWorkaround
Jul 7, 2026
Merged

BUILD-11815: Remove SIGNING_PROFILE workaround from release pipeline#473
okorach-sonar merged 1 commit into
mainfrom
chore/mmolnar/BUILD-11815-removeSigningProfileWorkaround

Conversation

@matemoln

@matemoln matemoln commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Remove the workflow-level SIGNING_PROFILE env var and explicit signing-profile input from the Windows Authenticode signing step
  • Rely on gh-action_azure-artifact-signing v1.1.0 auto-detection, which now treats main as a release branch alongside master and branch-*

The workaround was added because the action previously only auto-selected the release profile on master. With BUILD-11815 shipped in the action, manual profile selection is no longer needed.

Test plan

  • Merge and verify sign-windows succeeds on a main push with release OIDC credentials
  • Confirm the action logs Using signing profile 'release' without an explicit signing-profile input

gh-action_azure-artifact-signing v1.1.0 auto-detects release signing on
main and branch-* pushes, so the workflow no longer needs to override
signing-profile manually.
@matemoln
matemoln requested a review from a team as a code owner July 6, 2026 17:42
@gitar-bot

gitar-bot Bot commented Jul 6, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

Removes the redundant SIGNING_PROFILE workaround in favor of auto-detection provided by gh-action_azure-artifact-signing v1.1.0. 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 the behavior for this request:

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

Was this helpful? React with 👍 / 👎 | Gitar

@sonarqubecloud

sonarqubecloud Bot commented Jul 6, 2026

Copy link
Copy Markdown

@okorach-sonar okorach-sonar 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

@okorach-sonar
okorach-sonar merged commit 5f20926 into main Jul 7, 2026
10 checks passed
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