Repository navigation
Revert branch freeze PowerShell automation - #14692
Conversation
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
|
Can you please provide some context? EDIT: Chatted offline |
There was a problem hiding this comment.
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
.gitattributesLF 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.
There was a problem hiding this comment.
✅ 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.mdcorrectly removed alongside the code it documented.documentation/README.mdreference 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
Reverts #14495 and #14470.
No other changes.