Skip to content

style: migrate to oxlint/oxfmt - #250

Open
Tomahawk2191 wants to merge 7 commits into
mainfrom
isaac/migrate-to-oxlint-oxfmt
Open

style: migrate to oxlint/oxfmt#250
Tomahawk2191 wants to merge 7 commits into
mainfrom
isaac/migrate-to-oxlint-oxfmt

Conversation

@Tomahawk2191

@Tomahawk2191 Tomahawk2191 commented Sep 5, 2026

Copy link
Copy Markdown
Collaborator

What did I do?

Added multiple Oxlint and Oxfmt rules to enforce best practices for code quality. Replaces the prettier config that wasn't adopted fully throughout the repo.

Also added lint-staged rules which enforce the linter on any commits matching the config's wildcard. (see package.json)

Oxlint rules reference: https://oxc.rs/docs/guide/usage/linter/rules.html
Oxfmt rules reference: https://oxc.rs/docs/guide/usage/formatter/config-file-reference.html
"anti-slop" rules: https://github.com/dmmulroy/anti-slop

How did I test it?

Describe in detail steps you used to test the changes you have made.

Required checks:

  • Did you conduct a self-review?
  • Have you written unit or integration tests?

What could go wrong in the future? What parts of your code should the reviewer pay the most attention to?

Describe aspects of the PR that may become problems in the future.

Additional Comments for the Reviewers


Stack created with GitHub Stacks CLIGive Feedback 💬

@fraander

fraander commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

👷 Deploy Preview for sprightly-manatee-243873 processing.

Name Link
🔨 Latest commit 481bdcc
🔍 Latest deploy log https://app.netlify.com/projects/sprightly-manatee-243873/deploys/6aa5a519b60e9e000814d601

@Tomahawk2191
Tomahawk2191 force-pushed the isaac/migrate-to-oxlint-oxfmt branch from 6d0d020 to 1be92f8 Compare September 10, 2026 22:19
@Tomahawk2191
Tomahawk2191 force-pushed the isaac/migrate-to-oxlint-oxfmt branch from 1be92f8 to 5668a6d Compare September 11, 2026 01:10
Base automatically changed from isaac/fix-env to main September 11, 2026 01:11
@Tomahawk2191
Tomahawk2191 force-pushed the isaac/migrate-to-oxlint-oxfmt branch from 5668a6d to f49d88a Compare September 11, 2026 01:11
@Tomahawk2191
Tomahawk2191 force-pushed the isaac/migrate-to-oxlint-oxfmt branch from f49d88a to 5dde665 Compare September 11, 2026 01: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.

2 participants