Skip to content

fix: split test:lint and test:lint:staged scripts#27

Closed
willfarrell wants to merge 2 commits intodevelopfrom
fix/lint-staged-script
Closed

fix: split test:lint and test:lint:staged scripts#27
willfarrell wants to merge 2 commits intodevelopfrom
fix/lint-staged-script

Conversation

@willfarrell
Copy link
Copy Markdown
Owner

Summary

  • test:lint was incorrectly running biome check --staged — it should lint the entire codebase, not just staged files
  • Added new test:lint:staged script for the staged-only variant (used by pre-commit hook)
  • Updated git:lint-staged to call test:lint:staged instead of test:lint

Resulting scripts

"git:lint-staged": "npm run test:lint:staged",
"test:lint": "biome check --no-errors-on-unmatched",
"test:lint:staged": "biome check --staged --no-errors-on-unmatched",

Test plan

  • Verify npm run test:lint lints all files (not just staged)
  • Verify npm run test:lint:staged lints only staged files
  • Verify pre-commit hook still works via git:lint-staged

@willfarrell willfarrell deleted the fix/lint-staged-script branch April 18, 2026 22:14
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