Skip to content

Commit d14a355

Browse files
committed
[CI][Docs] Align CI, docs and Dependabot with the Oxlint/Oxfmt switch
1 parent 6e73616 commit d14a355

6 files changed

Lines changed: 13 additions & 6 deletions

File tree

.gitattributes

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,13 +4,14 @@
44
/.editorconfig export-ignore
55
/.npmrc export-ignore
66
/.nvmrc export-ignore
7+
/.oxfmtrc.json export-ignore
8+
/.oxlintrc.json export-ignore
79
/.php-cs-fixer.dist.php export-ignore
810
/.symfony.bundle.yaml export-ignore
911
/AGENTS.md export-ignore
1012
/CLAUDE.md export-ignore
1113
/assets export-ignore
1214
/docs export-ignore
13-
/eslint.config.js export-ignore
1415
/package.json export-ignore
1516
/phpstan.dist.neon export-ignore
1617
/phpunit.xml.dist export-ignore

.github/dependabot.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,8 @@ updates:
2424
schedule:
2525
interval: 'weekly'
2626
allow:
27-
- dependency-name: '@antfu/eslint-config'
28-
- dependency-name: 'eslint'
27+
- dependency-name: 'oxlint'
28+
- dependency-name: 'oxfmt'
2929
- dependency-name: '@rspack/core'
3030
- dependency-name: '@types/node'
3131
- dependency-name: 'jsdom'

.github/workflows/ci.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,8 @@ jobs:
3636
run: pnpm install --frozen-lockfile
3737
- name: Lint
3838
run: pnpm run lint
39+
- name: Format check
40+
run: pnpm run fmt:check
3941

4042
test:
4143
runs-on: ${{ matrix.os }}

AGENTS.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,11 +14,12 @@ The repo is a **Composer bundle** (`symfony/reprise`, PHP `src/`/`tests/` at the
1414

1515
## Commands
1616

17-
Package manager is **pnpm** (enforced via `packageManager` field). Node 22 (`.nvmrc`). The root `pnpm build`/`dev`/`test`/`lint` scripts run from the workspace root; `build`/`dev`/`test` delegate to the `assets` package, while `lint` runs at the root over `assets/**`.
17+
Package manager is **pnpm** (enforced via `packageManager` field). Node 22 (`.nvmrc`). The root `pnpm build`/`dev`/`test`/`lint`/`fmt` scripts run from the workspace root; `build`/`dev`/`test` delegate to the `assets` package, while `lint` (Oxlint) and `fmt` (Oxfmt) run at the root over the whole repo.
1818

1919
- `pnpm build` — delegates to `assets`, build via `tsdown` (bundles every `assets/src/*.ts` to `assets/dist/`)
2020
- `pnpm dev` — delegates to `assets`, `tsdown -w`, watch/rebuild
21-
- `pnpm lint``eslint .` at the root (@antfu/eslint-config, flat config), scoped to `assets/**`; `playground/` is ignored (fixture app, not library source)
21+
- `pnpm lint``oxlint` at the root (config in `.oxlintrc.json`); `pnpm lint:fix` auto-fixes. `playground/`, `assets/test/fixtures/` and `docs/` are ignored (not library source)
22+
- `pnpm fmt` / `pnpm fmt:check``oxfmt` at the root (config in `.oxfmtrc.json`); `fmt:check` is the read-only variant CI runs
2223
- `pnpm test` — delegates to `assets`, run tests (vitest, scoped to `assets/test/` via `vitest.config.ts` so it never picks up `.references/` clones)
2324
- `pnpm vitest run assets/test/index.test.ts` — run a single test file
2425
- `pnpm vitest run -t "hi vitest"` — run a single test by name

CONTRIBUTING.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -76,6 +76,9 @@ $ pnpm test
7676

7777
# Lint
7878
$ pnpm lint
79+
80+
# Format (use `pnpm fmt:check` to only check, without writing)
81+
$ pnpm fmt
7982
```
8083

8184
For manual end-to-end verification against a real Symfony backend, use the `playground/` app (a full Symfony 7 project that imports the plugin directly from `assets/`):

assets/tsconfig.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,5 +9,5 @@
99
"strictNullChecks": true,
1010
"esModuleInterop": true
1111
},
12-
"exclude": ["dist", "eslint.config.js"]
12+
"exclude": ["dist"]
1313
}

0 commit comments

Comments
 (0)