You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
.github/workflows/ci.yml runs pnpm turbo run test as one of its three steps. It executes zero tasks:
WARNING No tasks were executed as part of this run.
Tasks: 0 successful, 0 total
The root package.json defines "test": "turbo run test", but no package in the repository defines a test script for turbo to find, so the step passes by having nothing to do.
Why it is worth recording
This is a step that has never failed and, as configured, never can. A green step that cannot go red is indistinguishable in CI from one that is protecting something — and it is read as the latter, which is the whole cost. Every PR in this repository has been merged with a "test" job reporting success.
It is genuinely low urgency: nothing is broken, and there may be no unit-testable surface here — the repository is a Fumadocs site plus two node scripts. type-check and build are the checks that actually bite, and the Translations job now adds two more.
What the fix probably is
One of two calls, and it is a call rather than a transcription, which is why this is filed rather than fixed:
Delete the step. Honest, and removes a signal nobody should be reading. Cheapest.
Give it something to run. There is now a natural candidate: .github/scripts/check-translations.mjs and check-translation-ownership.mjs are pure functions over a checkout, and Enforce the translation output checklist as an automated check, not a prose list #67 is adding a third script with a self-test. Wiring that self-test in would make the step mean something.
⛔ Do not resolve this by adding a placeholder test that asserts true. That converts a step which is honestly empty into one that is dishonestly green.
Observed while reviewing #69, filed unassigned.
.github/workflows/ci.ymlrunspnpm turbo run testas one of its three steps. It executes zero tasks:The root
package.jsondefines"test": "turbo run test", but no package in the repository defines atestscript for turbo to find, so the step passes by having nothing to do.Why it is worth recording
This is a step that has never failed and, as configured, never can. A green step that cannot go red is indistinguishable in CI from one that is protecting something — and it is read as the latter, which is the whole cost. Every PR in this repository has been merged with a "test" job reporting success.
It is genuinely low urgency: nothing is broken, and there may be no unit-testable surface here — the repository is a Fumadocs site plus two node scripts.
type-checkandbuildare the checks that actually bite, and theTranslationsjob now adds two more.What the fix probably is
One of two calls, and it is a call rather than a transcription, which is why this is filed rather than fixed:
.github/scripts/check-translations.mjsandcheck-translation-ownership.mjsare pure functions over a checkout, and Enforce the translation output checklist as an automated check, not a prose list #67 is adding a third script with a self-test. Wiring that self-test in would make the step mean something.⛔ Do not resolve this by adding a placeholder test that asserts
true. That converts a step which is honestly empty into one that is dishonestly green.Related: #67.