Skip to content

Conversation

@thekatze
Copy link
Member

@thekatze thekatze commented Jul 16, 2025

Adds prettier to the ./server project to automatically format the typescript code.

This will make CI fail due to the new formatting rules until the codebase is formatted.
Holding off on actually formatting all of the code until #8 is merged to prevent running into formatting merge conflicts.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds Prettier configuration to the server project to enforce consistent TypeScript code formatting. The changes integrate Prettier with the existing ESLint setup and provide npm scripts for linting and automatic code formatting.

Key changes include:

  • Added Prettier configuration with specific formatting rules (4-space tabs, single quotes, no semicolons)
  • Updated package.json scripts to include Prettier checks in the lint process
  • Integrated eslint-config-prettier to prevent conflicts between ESLint and Prettier rules

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.

File Description
server/prettier.config.js New Prettier configuration file defining formatting rules
server/package.json Updated lint scripts and added Prettier dependencies
server/eslint.config.js Added eslint-config-prettier to prevent ESLint/Prettier conflicts
server/.prettierignore New ignore file excluding Rust code, build artifacts, and dependencies
Files not reviewed (1)
  • server/pnpm-lock.yaml: Language not supported

@thekatze thekatze changed the title set up prettier to automatically format javascript code set up prettier to automatically format typescript code Jul 16, 2025
@thekatze thekatze merged commit f3a9ff1 into main Jul 17, 2025
1 check failed
@thekatze thekatze deleted the format-js branch July 17, 2025 15: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.

3 participants