Repository navigation
fix(workflows): make managed YAML Prettier-stable - #594
Conversation
|
Review on The six locally reported APM temp-realpath failures are therefore not a regression introduced by this PR; GitHub's full test step is green on the same head. Remaining work is repository coordination only: this branch is now behind current Devtools main, so rebase it, resolve any generated-workflow overlap against current main, rerun CI, then mark ready for review. No additional functional change is required by this review. |
|
Review status: implementation direction is clean. Formatting through the published Devtools Prettier config is the right generic fix, and the sync -> Prettier-stable -> second-sync-unchanged regression covers the actual steady-state contract. Keep this PR Draft for now. Its previously reported six full-suite failures are exactly the APM temporary-root issue now fixed by PR #604. After #604 lands, rebase/update this branch and rerun the full |
92cb5c0 to
51ca992
Compare
51ca992 to
073a0e3
Compare
Summary
Validation
bun test src/tools/workflows/index.test.ts src/tools/workflows/renderWorkflowAsync.test.tsbun run buildbun run check-typesbun run lint(42 pre-existing warnings, no errors)bun run knip:checkbun run changeset:statusbun run formatandbun run format:checkbun run testhas six unrelated APM packed-fixture failures caused by path-containment rejection for files beneath temporary unpacked fixture directories.Changeset: patch for generated managed workflow output.
Closes #593