Skip to content

Decouple Prettier from ESLint - #8204

Draft
gonzaloriestra wants to merge 1 commit into
mainfrom
gonzalo/oxlint-01-prettier
Draft

Decouple Prettier from ESLint#8204
gonzaloriestra wants to merge 1 commit into
mainfrom
gonzalo/oxlint-01-prettier

Conversation

@gonzaloriestra

@gonzaloriestra gonzaloriestra commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Why

This is the first step of the ESLint-to-Oxlint migration. It separates formatting from linting so later PRs can change the linter without changing formatting behavior.

What changed

  • add standalone prettier and prettier:fix scripts for the source files currently linted
  • run formatting explicitly in local lint scripts and GitHub Actions
  • format GraphQL codegen output directly with Prettier
  • remove the prettier/prettier ESLint rule while retaining compatibility rules that prevent formatter conflicts
  • move Prettier to the workspace root and remove the direct eslint-plugin-prettier dependency
  • document the standalone formatting commands

Validation

  • pnpm install --frozen-lockfile
  • pnpm prettier
  • pnpm nx run-many --all --skip-nx-cache --target=lint --output-style=stream
  • verified eslint --print-config does not contain prettier/prettier

gonzaloriestra commented Jul 30, 2026

Copy link
Copy Markdown
Contributor Author

@github-actions github-actions Bot added the no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users. label Jul 30, 2026
@gonzaloriestra
gonzaloriestra force-pushed the gonzalo/oxlint-01-prettier branch 2 times, most recently from e5e7a99 to 8774cb9 Compare July 30, 2026 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant