-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Closed as not planned
Description
Note
The pull request "chore: Clean up lint and format script names" was created by @logaretm but did not reference an issue. Therefore this issue was created for better visibility in external tools like Linear.
Summary
Standardize lint/format script naming across the monorepo (53 files). Removes redundant/confusing scripts and makes naming consistent.
New root-level scripts
| Script | Command | Purpose |
|---|---|---|
check |
run-s format:check lint |
Read-only: format check + lint |
fix |
run-s format lint:fix |
Write: format + lint fix |
lint |
oxlint . --type-aware |
Lint only |
lint:fix |
oxlint . --fix --type-aware |
Lint + fix only |
format |
oxfmt . --write |
Format only |
format:check |
oxfmt . --check |
Format check only |
What changed
lintnow runs only oxlint (previously also ran oxfmt check)lint:fixreplaces oldfixfor oxlint auto-fix- New
checkruns bothformat:check+lint(replaces oldlintbehavior) - New
fixruns bothformat+lint:fix - All oxlint commands consistently include
OXLINT_TSGOLINT_DANGEROUSLY_SUPPRESS_PROGRAM_DIAGNOSTICSflag and--type-awareacross root and all sub-packages - Removes redundant scripts:
fix:oxlint,fix:oxfmt,lint:oxfmt,lint:oxlint - Updates CI workflow (
build.yml) to use new script names
Test plan
- CI lint job passes with
yarn lintinstead ofyarn lint:oxlint - CI format check job passes (unchanged
yarn format:check) -
yarn checkruns both format check and lint at root level -
yarn fixruns both format and lint fix at root level
🤖 Generated with Claude Code
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels