Skip to content

chore: include tests in prettier formatting#62

Merged
freema merged 1 commit intomainfrom
chore/prettier-tests
Mar 31, 2026
Merged

chore: include tests in prettier formatting#62
freema merged 1 commit intomainfrom
chore/prettier-tests

Conversation

@freema
Copy link
Copy Markdown
Collaborator

@freema freema commented Mar 31, 2026

Summary

  • Extend format and format:check npm scripts to include tests/**/*.ts
  • Apply prettier to all existing test files (21 files touched, style-only changes)

Follow-up to the discussion in #59.

Test plan

  • npx prettier --check "src/**/*.ts" "tests/**/*.ts" passes
  • No logic changes, only whitespace/formatting

Extend format and format:check scripts to cover tests/**/*.ts and
apply prettier to all existing test files.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@freema freema requested a review from juliandescottes March 31, 2026 18:20
@freema freema merged commit 7bad50e into main Mar 31, 2026
1 check passed
@juliandescottes
Copy link
Copy Markdown
Collaborator

Thanks for changing this. It's better to do it as early as possible. IMO it doesn't really matter whether we deal with test or "source" files. As long as they are files which can be linted/prettified and are going to be updated by more than 1 person, it's going to be useful in the long run.

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.

2 participants