style(oxlint/plugins): format test fixtures#15600
Conversation
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
1340328 to
65e1478
Compare
There was a problem hiding this comment.
Pull Request Overview
This PR updates formatting for Oxlint plugin test fixtures by enabling formatting for apps/oxlint/test/fixtures directory while maintaining exclusions for specific test files that depend on unformatted code for correctness.
Key Changes:
- Updated
oxfmtrc.jsoncto selectively format oxlint test fixtures while excluding other fixture directories and two specific test cases - Applied consistent formatting to 14 plugin test files including arrow function parameters, string concatenation, and object properties
- Updated test snapshots to match formatted test files
Reviewed Changes
Copilot reviewed 14 out of 15 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| oxfmtrc.jsonc | Refined ignore patterns to allow formatting of oxlint plugins while excluding other fixtures and specific test files |
| apps/oxlint/test/fixtures/utf16_offsets/plugin.ts | Reformatted multi-line string concatenations |
| apps/oxlint/test/fixtures/sourceCode_scope_methods/plugin.ts | Added parentheses to arrow function parameters |
| apps/oxlint/test/fixtures/sourceCode_late_access_after_only/plugin.ts | Reformatted multi-line message strings |
| apps/oxlint/test/fixtures/sourceCode_late_access/plugin.ts | Reformatted multi-line string concatenations across multiple functions |
| apps/oxlint/test/fixtures/sourceCode/plugin.ts | Reformatted multi-line string concatenations in create and createOnce rules |
| apps/oxlint/test/fixtures/settings/plugin.ts | Added parentheses to arrow function parameter |
| apps/oxlint/test/fixtures/selector/plugin.ts | Reformatted multi-line map/join chain for better readability |
| apps/oxlint/test/fixtures/selector/output.snap.md | Updated snapshot to reflect formatted test file |
| apps/oxlint/test/fixtures/selector/files/index.js | Removed unnecessary parentheses from arrow function expression |
| apps/oxlint/test/fixtures/scope_manager/plugin.ts | Added parentheses to arrow function parameters and reformatted multi-line strings |
| apps/oxlint/test/fixtures/parent/plugin.ts | Reformatted multi-line expressions and improved TypeScript error suppression comment |
| apps/oxlint/test/fixtures/isSpaceBetween/plugin.ts | Fixed indentation in context.report block |
| apps/oxlint/test/fixtures/fixes/plugin.ts | Removed unnecessary quotes from object property name |
| apps/oxlint/test/fixtures/estree/plugin.ts | Reformatted multi-line strings and added parentheses to arrow function parameters |
| apps/oxlint/test/fixtures/comments/plugin.ts | Reformatted multi-line string concatenations |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Merge activity
|
Format `oxlint` test fixtures. The directory `fixtures` contains not just the fixtures themselves, but also Oxlint plugins, which should be formatted. Skip 2 specific test fixture files which must not be formatted to maintain correctness of the tests.
65e1478 to
aee9be1
Compare

Format
oxlinttest fixtures. The directoryfixturescontains not just the fixtures themselves, but also Oxlint plugins, which should be formatted.Skip 2 specific test fixture files which must not be formatted to maintain correctness of the tests.