Skip to content

Revert branch freeze PowerShell automation - #14692

Merged
AlesProkop merged 2 commits into
dotnet:mainfrom
AlesProkop:dev/alesprokop/revert-14495-14470
Aug 12, 2026
Merged

AlesProkop merged 2 commits into
dotnet:mainfrom
AlesProkop:dev/alesprokop/revert-14495-14470

Conversation

@AlesProkop

Copy link
Copy Markdown
Member

Reverts #14495 and #14470.

No other changes.

AlesProkop and others added 2 commits August 12, 2026 09:29
This reverts commit 0567456.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 42691a11-7401-40fe-b781-87654fed03fa
This reverts commit 317c30b.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 42691a11-7401-40fe-b781-87654fed03fa
Copilot AI lite review requested due to automatic review settings August 12, 2026 07:30
@AlesProkop
AlesProkop requested a review from a team as a code owner August 12, 2026 07:30
@ViktorHofer

ViktorHofer commented Aug 12, 2026 •

Copy link
Copy Markdown
Member

Can you please provide some context? EDIT: Chatted offline

Copilot AI 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.

Pull request overview

This PR reverts the previously introduced branch-freeze (merge-freeze) PowerShell-based GitHub Actions automation and its supporting documentation, restoring the repository to the pre-#14470/#14495 state.

Changes:

  • Removed the branch-freeze GitHub Actions workflows (command handling, per-PR status stamping, bulk refresh, and test workflow).
  • Removed the branch-freeze PowerShell implementation (workflows scripts, modules, and unit-test harness) and the checked-in allowlist.
  • Removed branch-freeze documentation and the documentation index link, plus the now-unneeded .gitattributes LF rules for those scripts.

Reviewed changes

Copilot reviewed 23 out of 23 changed files in this pull request and generated no comments.

Show a summary per file
File Description
documentation/README.md Removes the documentation index entry for branch-freeze.
documentation/branch-freeze.md Removes the branch-freeze feature documentation page.
.github/workflows/branch-freeze-tests.yml Removes the workflow that validated branch-freeze PowerShell syntax and ran unit tests.
.github/workflows/branch-freeze-refresh.yml Removes the workflow that bulk-refreshed branch-freeze commit statuses on open PRs.
.github/workflows/branch-freeze-pr-status.yml Removes the workflow that evaluated/stamped branch-freeze status for individual PR events.
.github/workflows/branch-freeze-command.yml Removes the /freeze and /unfreeze command processing workflow.
.github/branch-freeze/workflows/set-pr-status.ps1 Removes the script that posted branch-freeze status for a single PR head SHA.
.github/branch-freeze/workflows/refresh-pr-statuses.ps1 Removes the script that iterated open PRs and refreshed branch-freeze statuses.
.github/branch-freeze/workflows/is-allowed.ps1 Removes the allowlist authorization script for command execution.
.github/branch-freeze/workflows/handle-command.ps1 Removes the script that parsed and executed /freeze//unfreeze and replied/reacted.
.github/branch-freeze/workflows/deny-command.ps1 Removes the script that denied unauthorized freeze/unfreeze commands.
.github/branch-freeze/tests/run-tests.ps1 Removes the PowerShell unit test harness for branch-freeze logic/security behaviors.
.github/branch-freeze/tests/mock-gh.ps1 Removes the gh CLI mock used by the branch-freeze unit tests.
.github/branch-freeze/components/issue-comments/BranchFreezeCommentParser.psm1 Removes command/issue-body parsing module used by branch-freeze automation.
.github/branch-freeze/components/issue-comments/BranchFreezeCommentComposer.psm1 Removes reply/issue-body/status-description composition module for branch-freeze automation.
.github/branch-freeze/components/github/GitHubStatusChecksClient.psm1 Removes the wrapper used to post commit statuses via gh api.
.github/branch-freeze/components/github/GitHubRepositoryClient.psm1 Removes repo/branch helper wrapper used by branch-freeze automation.
.github/branch-freeze/components/github/GitHubPullRequestsClient.psm1 Removes PR enumeration wrapper used by branch-freeze refresh logic.
.github/branch-freeze/components/github/GitHubIssuesClient.psm1 Removes issue/label/comment/reaction wrappers used by branch-freeze automation.
.github/branch-freeze/components/github/GitHubCli.psm1 Removes the gh invocation wrapper with retry logic used by branch-freeze components.
.github/branch-freeze/components/BranchFreeze.psm1 Removes the core domain module managing tracking issues and freeze state.
.github/branch-freeze-allowlist.txt Removes the checked-in allowlist of accounts permitted to run freeze/unfreeze.
.gitattributes Removes the LF line-ending rules that were specific to the deleted branch-freeze PowerShell scripts.

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@github-actions github-actions Bot 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.

✅ 24/24 dimensions clean — no findings.

This is a pure revert PR (reverts #14495 and #14470) that deletes CI automation infrastructure only — PowerShell branch-freeze workflows, scripts, tests, allowlist, and documentation. There are no C# source code changes.

All applicable dimensions were evaluated:

  • Build Infrastructure Care: CI/CD YAML files and PowerShell scripts removed cleanly. The revert is consistent and complete.
  • Documentation Accuracy: documentation/branch-freeze.md correctly removed alongside the code it documented. documentation/README.md reference removed.
  • Backwards Compatibility: No MSBuild build behavior change. The removed automation was GitHub Actions CI tooling, not part of the build engine.
  • Scope & PR Discipline: Single concern, clean revert, well-scoped.

Generated by Expert Code Review (on open) for #14692 · sonnet46 · 43.2 AIC · ⌖ 7.6 AIC · ⊞ 5.3K

@AlesProkop
AlesProkop merged commit 504051c into dotnet:main Aug 12, 2026
19 checks passed
This was referenced Oct 18, 2026

This branch was previously deployed

1 inactive deployment
copilot-pat-pool — be9236bd Deployed Aug 12, 2026 by AlesProkop via conclusion #645
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