Skip to content

build: switch to oxfmt and oxlint - #16

Merged
TheGreatAxios merged 2 commits into
cl-embedding-e2e-test-layoutfrom
cl-9371-embedding-switch-to-oxfmt-and-oxlint
Sep 27, 2026
Merged

TheGreatAxios merged 2 commits into
cl-embedding-e2e-test-layoutfrom
cl-9371-embedding-switch-to-oxfmt-and-oxlint

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Sep 26, 2026 •

Copy link
Copy Markdown
Contributor

Closes CL-9371

Summary

  • Remove ESLint (eslint, @eslint/js, typescript-eslint, jiti, eslint.config.ts) and Prettier (prettier, .prettierrc.json)
  • Add oxfmt and oxlint as exact-pinned devDependencies
  • CI runs bun run format:check in place of bunx prettier --check .; the existing bun run lint step now runs oxlint
  • IMPLEMENTATION.md: dev dependency list names oxlint and oxfmt instead of ESLint and Prettier
  • .oxfmtrc.json sets printWidth: 80 (Prettier's width); everything else is oxfmt's defaults, including sorting package.json keys
  • .oxlintrc.json turns the correctness category from warn to error; no rules are disabled
  • lint runs oxlint, format runs oxfmt, format:check runs oxfmt --check

Stacked on #14. This supersedes the ESLint setup from #12, which stays open below it in the stack.

Commits

  1. build: switch to oxfmt and oxlint: tooling, configs, CI and docs
  2. style: format with oxfmt: mechanical reformat only (1 file changed, 21 insertions(+), 21 deletions(-)), safe to skip in review

@TheGreatAxios
TheGreatAxios force-pushed the cl-9371-embedding-switch-to-oxfmt-and-oxlint branch from 9b4b17f to 974244c Compare September 26, 2026 02:51
@TheGreatAxios
TheGreatAxios changed the base branch from main to cl-embedding-e2e-test-layout September 26, 2026 02:51
@TheGreatAxios
TheGreatAxios added this pull request to stack #15 September 26, 2026 02:52
@TheGreatAxios
TheGreatAxios merged commit f6c73ed into main Sep 27, 2026
1 check passed
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