Skip to content

Use oxlint and prettier for linting and formatting - #246

Merged
radubahmata merged 14 commits into
masterfrom
devex/lint-fmt
Apr 7, 2026
Merged

radubahmata merged 14 commits into
masterfrom
devex/lint-fmt

Conversation

@radubahmata

@radubahmata radubahmata commented Apr 5, 2026 •

Copy link
Copy Markdown
Contributor

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.

@radubahmata
radubahmata requested a review from a team as a code owner April 5, 2026 21:55
@radubahmata

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.

@radubahmata radubahmata changed the title [DRAFT] Use oxlint linter and prettier formatter Use oxlint and prettier for linting and formatting Apr 6, 2026
@radubahmata
radubahmata requested review from moodmosaic and wileyj April 6, 2026 20:19

@wileyj wileyj left a comment

Copy link
Copy Markdown
Contributor

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

@radubahmata
radubahmata merged commit 9becb32 into master Apr 7, 2026
18 checks passed
@radubahmata
radubahmata 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