Skip to content

fix: format files flagged by prettier - #25

Open
liviaUeno wants to merge 3 commits into
developfrom
fix-prettier-formatting
Open

liviaUeno wants to merge 3 commits into
developfrom
fix-prettier-formatting

Conversation

@liviaUeno

Copy link
Copy Markdown

Context

The Static Analysis / Prettier check (pnpm lint:prettier, i.e. prettier --check .) fails on develop because several files are not formatted.

What changed

  • Ran Prettier on the 37 files it flagged (apps/web sources, one unit test, infra/*.yaml, and two files in shared-components).
  • Formatting only: whitespace, line breaks, trailing commas and parentheses. No behavior change.

Left out on purpose

RoomListHeaderViewModel.ts and RoomListHeaderView.tsx are not included here. They are also modified in #23, which formats them, so including them in this PR would cause a merge conflict.

Verified

  • pnpm lint:prettier now reports only those two files.
  • git diff -w against develop shows 28 files with changes beyond whitespace. All of them are line joins or splits and trailing commas.

Notes

Other CI checks that already fail on develop (TypeScript, ESLint, etc.) are out of scope and tracked in separate cards.

Done when: Static Analysis / Prettier is green (after #23 is merged).

@liviaUeno
liviaUeno requested a review from zZMathSP October 15, 2026 15:41

This branch has not been deployed

No deployments
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