@mnci/cli 1.11.1 → 1.11.2

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/README.md CHANGED
@@ -223,8 +223,11 @@ would make `npm run lint` and `npm run format:check` mutually unsatisfiable.
223
223
  Choosing Prettier means accepting its call there.
224
224
 
225
225
  `npm run lint` checks code quality; `npm run format` (write) and
226
- `npm run format:check` (CI-safe) handle formatting. The CI runs lint/test/build
227
- only — formatting is left as a local/pre-commit step.
226
+ `npm run format:check` (CI-safe) handle formatting. **Both run in CI**, as
227
+ separate steps: ESLint is configured for correctness only and deliberately
228
+ reports nothing about formatting, so Prettier needs a gate of its own or the
229
+ whole formatting opinion is advisory and a workspace drifts out of compliance
230
+ with no signal anywhere.
228
231
 
229
232
  TypeScript is not a question — every workspace runs the **dual compiler** from
230
233
  [Nx's TS 7 guide](https://nx.dev/docs/technologies/typescript/guides/typescript-7):
package/dist/cli.js CHANGED
@@ -1523,11 +1523,17 @@ steps:
1523
1523
  - script: npx nx sync:check
1524
1524
  displayName: Verify the workspace is synced (run 'npx nx sync' locally and commit if this fails)
1525
1525
 
1526
- # One verify for every run (PR and main). \`npm run lint\` abstracts the chosen
1527
- # linter (eslint via nx, or oxlint), so the pipeline needs no linter branch;
1528
- # Nx cache makes unchanged test/build projects instant.
1526
+ # One verify for every run (PR and main). Nx cache makes unchanged test/build
1527
+ # projects instant.
1529
1528
  - script: npm run lint
1530
1529
  displayName: Lint
1530
+
1531
+ # Prettier owns ALL formatting in this workspace, so it needs a gate of its
1532
+ # own: ESLint is configured for correctness only and deliberately reports
1533
+ # nothing about formatting. Without this step the entire formatting opinion is
1534
+ # advisory, and a workspace drifts out of compliance with no signal anywhere.
1535
+ - script: npm run format:check
1536
+ displayName: Check formatting (run 'npm run format' locally to fix)
1531
1537
  # \`lint\` here also runs any Nx lint targets \`npm run lint\` does not cover \u2014
1532
1538
  # notably Python's ruff (every Python project owns a hand-authored \`lint\`
1533
1539
  # target). For the eslint stack the JS lint runs twice, but Nx caches the
@@ -1694,11 +1700,18 @@ jobs:
1694
1700
  - run: npx nx sync:check
1695
1701
  name: Verify the workspace is synced (run 'npx nx sync' locally and commit if this fails)
1696
1702
 
1697
- # One verify for every run (PR and main). \`npm run lint\` abstracts the chosen
1698
- # linter (eslint via nx, or oxlint), so the workflow needs no linter branch;
1699
- # Nx cache makes unchanged test/build projects instant.
1703
+ # One verify for every run (PR and main). Nx cache makes unchanged
1704
+ # test/build projects instant.
1700
1705
  - run: npm run lint
1701
1706
  name: Lint
1707
+
1708
+ # Prettier owns ALL formatting in this workspace, so it needs a gate of
1709
+ # its own: ESLint is configured for correctness only and deliberately
1710
+ # reports nothing about formatting. Without this step the entire
1711
+ # formatting opinion is advisory, and a workspace drifts out of
1712
+ # compliance with no signal anywhere.
1713
+ - run: npm run format:check
1714
+ name: Check formatting (run 'npm run format' locally to fix)
1702
1715
  # \`lint\` here also runs any Nx lint targets \`npm run lint\` does not cover \u2014
1703
1716
  # notably Python's ruff (every Python project owns a hand-authored \`lint\`
1704
1717
  # target). For the eslint stack the JS lint runs twice, but Nx caches the