Skip to content

Chore/add format check to ci#112

Open
thisisnkc wants to merge 3 commits intoJulien-R44:mainfrom
thisisnkc:chore/add-format-check-to-ci
Open

Chore/add format check to ci#112
thisisnkc wants to merge 3 commits intoJulien-R44:mainfrom
thisisnkc:chore/add-format-check-to-ci

Conversation

@thisisnkc
Copy link
Contributor

Description

Add Prettier format checking to the CI workflow to ensure consistent code formatting across the project.

Changes

  • Added a new format job to the GitHub Actions CI workflow
  • The job runs pnpm format to validate that all code follows the project's Prettier configuration
  • Runs on every push and pull request, alongside existing lint, typecheck, build, and test jobs

Why

This ensures code formatting consistency is enforced at the CI level, preventing formatting-related issues from being merged and maintaining a uniform code style across the codebase.

Testing

The format check job will validate all code changes against Prettier's configured rules. Developers can run pnpm format locally before pushing to ensure compliance. If found any formatting issue(s), they can run pnpm format --write

- Add format job to GitHub Actions workflow
- Runs pnpm format:check to validate code formatting
- Ran the formatter once across the entire codebase to ensure all existing files comply with Prettier rules, preventing CI failures on the initial run
@changeset-bot
Copy link

changeset-bot bot commented Mar 25, 2026

⚠️ No Changeset found

Latest commit: 2c0a5cf

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

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.

1 participant