Skip to content

build: switch to oxfmt and oxlint - #34

Open
TheGreatAxios wants to merge 2 commits into
cl-9336-artifacts-store-upload-bytes-per-version-so-versionn-returnsfrom
cl-9379-artifacts-switch-to-oxfmt-and-oxlint
Open

TheGreatAxios wants to merge 2 commits into
cl-9336-artifacts-store-upload-bytes-per-version-so-versionn-returnsfrom
cl-9379-artifacts-switch-to-oxfmt-and-oxlint

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Sep 26, 2026 •

Copy link
Copy Markdown
Contributor

Closes CL-9379

Summary

  • Stacked on fix(uploads): store upload bytes per version #31, the tip of the open stack, so the reformat covers every file on it
  • Add oxfmt and oxlint as exact-pinned devDependencies (the repo had no formatter or linter)
  • Run lint and format:check in the test workflow after install
  • Remove the unused ArtifactDb type import from src/content-store.ts and an unused row binding in e2e/tools.test.ts (no-unused-vars)
  • .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

Commits

  1. build: switch to oxfmt and oxlint: tooling, configs and lint fixes
  2. style: format with oxfmt: mechanical reformat only (28 files changed, 1487 insertions(+), 520 deletions(-)), safe to skip in review

@TheGreatAxios
TheGreatAxios force-pushed the cl-9379-artifacts-switch-to-oxfmt-and-oxlint branch from d8c761b to fd7e66f Compare September 26, 2026 03:15
@TheGreatAxios
TheGreatAxios changed the base branch from main to cl-9336-artifacts-store-upload-bytes-per-version-so-versionn-returns September 26, 2026 03:15
@TheGreatAxios
TheGreatAxios added this pull request to stack #32 September 26, 2026 03:15
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