Skip to content

Use oxlint and prettier for linting and formatting#246

Merged
BowTiedRadone merged 14 commits intomasterfrom
devex/lint-fmt
Apr 7, 2026
Merged

Use oxlint and prettier for linting and formatting#246
BowTiedRadone merged 14 commits intomasterfrom
devex/lint-fmt

Conversation

@BowTiedRadone
Copy link
Copy Markdown
Contributor

@BowTiedRadone BowTiedRadone commented Apr 5, 2026

This PR adds linting and formatting to Rendezvous. oxlint enforces correctness, suspicious code, performance, and style rules, while prettier with @trivago/prettier-plugin-sort-imports handles code formatting and import ordering.

Key additions

  • oxlint linter with tuned rule categories (correctness, suspicious, perf, style) and project-specific overrides
  • prettier formatter with sorted imports
  • CI pipeline: fmt-lint job runs first as a gate for other jobs
  • Codebase-wide fixes: shadowed declarations, consistent type definitions (type over interface), arrow function expressions, Record<K, V> over index signatures, required params before optional

New scripts

  • lint: run oxlint
  • lint:fix: run oxlint and fix (applicable for rules that can be fixed automatically)
  • fmt: run prettier
  • fmt:check: check formatting without writing

Closes #194.

@BowTiedRadone BowTiedRadone requested a review from a team as a code owner April 5, 2026 21:55
@BowTiedRadone
Copy link
Copy Markdown
Contributor Author

What's left here is to carefully go over the currently disabled linting rules and re-enable the low noise/high value ones.

@BowTiedRadone BowTiedRadone changed the title [DRAFT] Use oxlint linter and prettier formatter Use oxlint and prettier for linting and formatting Apr 6, 2026
Copy link
Copy Markdown
Contributor

@wileyj wileyj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is definitely linted! seems fine, and tests are all passing

@BowTiedRadone BowTiedRadone merged commit 9becb32 into master Apr 7, 2026
18 checks passed
@BowTiedRadone BowTiedRadone deleted the devex/lint-fmt branch April 7, 2026 21:54
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.

Setup CI linter and formatter for code standards enforcement

2 participants