Skip to content

Conversation

seratch
Copy link
Member

@seratch seratch commented Oct 3, 2025

This pull request changes the prettier configuration:

  • consistently use 2 spaces for indentation like commonly done
  • apply the formatter to config files too

Once #7 is merged, I will run npm run format and commit the changes to this PR.

Copy link
Collaborator

@steven10a steven10a left a comment

Choose a reason for hiding this comment

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

LGTM

@gabor-openai gabor-openai requested a review from Copilot October 3, 2025 20:59
Copy link

@Copilot 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 standardizes the Prettier configuration to align with common JavaScript/TypeScript formatting practices. The changes establish consistent formatting rules and expand the formatter scope to include configuration files.

  • Updated Prettier configuration with standardized settings and added schema validation
  • Expanded format command to include configuration files (JSON, JS, etc.) beyond just TypeScript source files
  • Added .prettierignore file to exclude build artifacts and dependencies from formatting

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
package.json Updated format script to include config files and various JS/TS extensions
.prettierrc Reorganized config with schema, changed arrow parens to "always", and added endOfLine setting
.prettierignore Added ignore patterns for build outputs and dependencies

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@gabor-openai gabor-openai merged commit 0a662dc into main Oct 3, 2025
@gabor-openai gabor-openai deleted the format branch October 6, 2025 20:14
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