Skip to content

fix(ci): make the lint command a read-only check - #189

Merged
ttbombadil merged 1 commit into
mainfrom
fix/ci-readonly-lint
Oct 5, 2026
Merged

ttbombadil merged 1 commit into
mainfrom
fix/ci-readonly-lint

Conversation

@ttbombadil

Copy link
Copy Markdown
Collaborator

Summary

Make the shared npm run lint command a read-only check by removing --fix. CI invokes this script; fixable errors previously passed after changing the checkout. Keep developer lint-staged autofixing.

Verification

  • Behavioral regression runs the actual script in a unique temporary ESLint fixture: fixable error must exit 1 and leave source unchanged. RED (old command exited 0) → GREEN.
  • Independent read-only review: no blocking findings.
  • Full unit/coverage, types, docs, lint, build and Sonar gates.
  • Real toolchain and Docker integration; full Chromium E2E using temporary port3307 adapter because an unrelated local project owns3000.
  • Normal commit/push hooks, no bypass.

Base: freshly fetched e92f5b466ec61cdf4ada1d78e901ac868885065e, after green post-merge CI for #188. No runtime, Provider, SSOT/ADR or Tutor-semantic changes; protected/untracked user files untouched.

@ttbombadil
ttbombadil merged commit 5b42587 into main Oct 5, 2026
5 checks passed
@ttbombadil
ttbombadil deleted the fix/ci-readonly-lint branch October 5, 2026 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant