Skip to content

chore(dev-deps): replace eslint+prettier with oxlint+oxfmt - #448

Merged
AriPerkkio merged 3 commits into
mainfrom
chore/oxlint-oxfmt
Sep 7, 2026
Merged

AriPerkkio merged 3 commits into
mainfrom
chore/oxlint-oxfmt

Conversation

@AriPerkkio

@AriPerkkio AriPerkkio commented Sep 1, 2026 •

Copy link
Copy Markdown
Member

Issue:

What Changed

Replaces eslint with oxlint and enables type-aware linting. Replaces prettier with oxfmt.

To review:

  • chore(dev-deps): replace eslint+prettier with oxlint+oxfmt 8cdfd8a

Feel free to ignore:

How to test

  • CI pass

@changeset-bot

changeset-bot Bot commented Sep 1, 2026 •

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: c29269e

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes changesets to release 2 packages
Name Type
@chromatic-com/cypress Minor
@chromatic-com/playwright Minor

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@AriPerkkio AriPerkkio mentioned this pull request Sep 1, 2026
12 of 18 tasks
@pkg-pr-new

pkg-pr-new Bot commented Sep 1, 2026 •

Copy link
Copy Markdown

Open in StackBlitz

pnpm add https://pkg.pr.new/@chromatic-com/cypress@c29269e -D
pnpm add https://pkg.pr.new/@chromatic-com/playwright@c29269e -D
pnpm add https://pkg.pr.new/@chromatic-com/vitest@c29269e -D

commit: c29269e

@AriPerkkio
AriPerkkio force-pushed the chore/oxlint-oxfmt branch 2 times, most recently from f9a30e1 to 6db5db8 Compare September 2, 2026 06:59
Comment thread .github/workflows/test-base.yml Outdated
Base automatically changed from feat/esm-only to main September 4, 2026 11:06
@AriPerkkio
AriPerkkio force-pushed the chore/oxlint-oxfmt branch 2 times, most recently from 4d92304 to 236a985 Compare September 4, 2026 12:02
@AriPerkkio
AriPerkkio marked this pull request as ready for review September 4, 2026 12:28
Comment thread eslint.config.mjs
Comment on lines -8 to -20
ignores: [
'**/node_modules/**',
'**/test-results/**',
'**/dist/**',
'**/.cache/**',
'**/*.d.ts',
'**/storybook-static/**',
'**/tsdown.config.ts',
'**/playwright.config.ts',
'**/tests/**',
'**/__playwright-tests__/**',
'test-server/fixtures/**',
],

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe you want (some of) these ignored by oxlint/oxfmt too?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

They read .gitignore automatically so most of them are automatically ignored. Rest are fine to be included (like Playwright config, tsdown config).

@AriPerkkio
AriPerkkio merged commit 011560c into main Sep 7, 2026
12 checks passed
@AriPerkkio
AriPerkkio deleted the chore/oxlint-oxfmt branch September 7, 2026 13:08
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.

2 participants