Skip to content

Pin sha separate workflows - #38036

Merged
BillWagner merged 2 commits into
dotnet:mainfrom
BillWagner:pin-sha-separate-workflows
Oct 28, 2026
Merged

BillWagner merged 2 commits into
dotnet:mainfrom
BillWagner:pin-sha-separate-workflows

Conversation

@BillWagner

Copy link
Copy Markdown
Member

This makes two related changes:

First, use the updated OPS preview workflow implemented in dotnet/docs-actions#37. This separates the status creation from the work that updates the PR descriptions. The security boundary between them means the first runs in the PR author's context, with minimal rights. The second is only triggered when the work completes and always runs in the base branch context to update the PR and description.

Second, pin all actions to the commit SHA.

Split the OPS status into two workflows. One generates the report, the second updates the PR description. The second runs in a different context and is the only workflows with write perms.
Pin the commit SHA for each workflow.
@BillWagner
BillWagner requested review from gewarren and a balanced review from Copilot and removed request for Copilot October 28, 2026 17:00
@BillWagner BillWagner closed this Oct 28, 2026
@BillWagner BillWagner reopened this Oct 28, 2026
@BillWagner BillWagner closed this Oct 28, 2026
@BillWagner BillWagner reopened this Oct 28, 2026
@BillWagner
BillWagner merged commit ee9227e into dotnet:main Oct 28, 2026
18 of 21 checks passed
@BillWagner
BillWagner deleted the pin-sha-separate-workflows branch October 28, 2026 18:29
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