Fix/remove trailing semicolon fixer - #455
Conversation
check_suite:[completed] and workflow_run:[completed] made this workflow fire once per CI completion on every watched workflow -- a multiplier on every unit of estate CI activity, and the mechanism behind the notification burst of 2026-07-20/21 (measured: 3,041 CheckSuite notifications inside a ~19-hour window, and none since). The workflow is currently disabled_manually, which is why the burst stopped. That makes the risk latent rather than resolved: these triggers plus contents:write / pull-requests:write mean re-enabling it would restart the multiplier. Removing them so re-enabling is safe. pull_request, pull_request_review and workflow_dispatch are retained -- the monitoring path the workflow was neutered down to in 2026-06 is untouched. Verified: YAML parses and the trigger map is now exactly pull_request, pull_request_review, workflow_dispatch. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
…em repos dogfood-gate.yml pinned hyperpolymath/a2ml-validate-action and hyperpolymath/k9-validate-action. Both repositories are DELETED (verified 404). An unresolvable uses: ref produces NO check run at all -- not a red one -- so 'Validate A2ML manifests' and 'Validate K9 contracts' could never report, and any ruleset requiring them was unsatisfiable. That is what blocked this very PR. The actions were not lost, they MOVED: hyperpolymath/a2ml-ecosystem/validate-action (action.yml verified at the pin) hyperpolymath/k9-ecosystem/validate-action (action.yml verified at the pin) Repointed rather than vendored. The estate handover proposed vendoring the scripts into ~272 consumer repos; that would create 272 copies to drift, which the same document names as this estate's most recurring failure mode. Repointing is one line per file and keeps a single source. SHAs are the ones already proven green in the-nash-equilibrium#83; both verified to resolve (2026-07-25) and to contain validate-action/action.yml at that exact ref. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
…g (fixes clippy compile error)
![gitar-bot[bot]](https://avatars.githubusercontent.com/in/827041?s=60&v=4){kind=small}
|
Note Automatic reviews are paused because your trial's included automatic processing has been used for this period. Upgrade now, or comment "Gitar review" to run a review anytime. Code Review ✅ Approved 1 resolved / 1 findingsRemoves a stray semicolon in the fixer to resolve a clippy compile error, but the file was corrupted with raw diff text instead of valid Rust. ✅ 1 resolved✅ Bug: fixer.rs replaced with raw diff text, not valid Rust
OptionsDisplay: compact → Showing less information. Comment with these commands to change the behavior for this request:
Was this helpful? React with 👍 / 👎 | Gitar |
1 participant
Summary
Closes #
Type of change
How has this been verified?
Checklist
git commit -S).SPDX-License-Identifier(code/configMPL-2.0,prose
CC-BY-SA-4.0); I did not relicense existing files.Notes for reviewers
Summary by Gitar
fixer.rssoexpand_templatereturnsStringand fixes clippy compile errorThis will update automatically on new commits.