Conversation
…nt block LFG main has comment improvements in low-memory.yml that didn't make it back to AceHack via #47 (which only backported the concurrency + timeout fixes). Close the residual drift: - Add push-to-main bullet at the top of "What this workflow does" - Add backstop parenthetical to schedule bullet - Add ubuntu-slim-was-removed-from-gate.yml note explaining the no- duplicate-run guarantee Per #43's diff-minimization invariant: AceHack and LFG should match on main with any drift rigorously accounted for. After this lands, \`git diff acehack/main..origin/main -- .github/workflows/low-memory.yml\` should return zero. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Aligns .github/workflows/low-memory.yml’s “What this workflow does” comment block with the latest wording from LFG main to restore AceHack↔LFG 0-diff on this workflow file.
Changes:
- Adds an explicit “push to main” bullet as the primary trigger.
- Clarifies the schedule trigger as a backstop (weekends / missed pushes).
- Documents why
ubuntu-slimwas removed fromgate.yml’s matrix to avoid duplicate runs.
…kflow does" bullet (Copilot P2) Copilot flagged the bullet's "every merge (primary trigger)" as technically imprecise — the trigger is push.branches:[main] which includes direct/force pushes if ever allowed. Match the more precise wording used in the header (line 4-6) per the suggestion. Now the bullet reads: push to main: runs on every push to main (in practice every merge; primary trigger). Same imprecise wording is on LFG main (came in via LFG Lucent-Financial-Group#644). Will forward-sync this fix to LFG via the next sibling sync to keep 0-diff. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Closes residual drift between AceHack and LFG main on
low-memory.yml. LFG Lucent-Financial-Group#644's review process added comment improvements that didn't make it back to AceHack via #47 (which only backported the concurrency + timeout fixes).What changed
Three comment additions to the "What this workflow does" block:
Why
Per #43's diff-minimization invariant: AceHack and LFG should match on main with any drift rigorously accounted for. After this lands,
git diff acehack/main..origin/main -- .github/workflows/low-memory.ymlreturns zero.This is the kind of post-sync cleanup that closes the loop on today's 5-PR cascade (#45/#46 forward-sync + LFG Lucent-Financial-Group#644 review fixes + #47 backport).
🤖 Generated with Claude Code