Skip to content

Conversation

@matux
Copy link
Contributor

@matux matux commented Oct 7, 2025

Description of the change

Prepares for codebase-wide formatting by modernizing Prettier configuration, matching Olympus/Fuji ruleset and idiosyncrasy.

Changes

  • Migrated prettier.config.cjsprettier.config.js (ESM)
  • Upgraded Prettier 3.2.5 → 3.6.2
  • Added npm run format and npm run format:check scripts
  • Exclude examples directory and fixture sources in test/fixtures to preserve sourcemaps based on the original files.

Related issues

@matux matux requested a review from Copilot October 7, 2025 14:58
@matux matux self-assigned this Oct 7, 2025
Copy link
Contributor

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

Modernizes the Prettier configuration by migrating from CommonJS to ESM format and upgrading to a newer version, along with adding formatting scripts to improve code consistency across the codebase.

  • Migrated Prettier config from CommonJS (module.exports) to ESM (export default) format
  • Upgraded Prettier from version 3.2.5 to 3.6.2
  • Added npm scripts for code formatting and format checking

Reviewed Changes

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

File Description
prettier.config.js Converted from CommonJS to ESM format and removed trailingComma setting
package.json Updated Prettier version and added format/format:check npm scripts
eslint.config.mjs Applied consistent formatting to rule definitions (removing quotes around property names)

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

@matux matux force-pushed the matux/eslint-prettier branch from 8fd042e to 312a88b Compare October 7, 2025 15:41
@matux matux changed the title Migrate Prettier config to modern ESM format Prepare Prettier config Oct 7, 2025
@matux matux merged commit 9c4222d into master Oct 7, 2025
4 checks passed
@matux matux deleted the matux/eslint-prettier branch October 7, 2025 15:48
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