Skip to content

chore: Upgrade prettier to 3.9.10 and format every file type - #1441

Open
marcleblanc2 wants to merge 2 commits into
remove-single-container-leftoversfrom
upgrade-prettier
Open

marcleblanc2 wants to merge 2 commits into
remove-single-container-leftoversfrom
upgrade-prettier

Conversation

@marcleblanc2

Copy link
Copy Markdown
Contributor

Stacked on #1433; merge that one first.

Changed

  • prettier 1.19.1 → 3.9.10, @sourcegraph/prettierrc 3.0.2 → 3.0.3 (adds arrowParens: avoid)
  • Deleted .prettierignore, which excluded .github/ and *.md
  • yarn prettier / yarn prettier-check now run on . instead of a js/md glob, so yaml and json are formatted too
  • Reformatted the 15 files prettier 3 flagged: mostly double → single quotes in yaml, dropped blank lines, and 2-space list indentation in release.yaml and docker-compose/examples/README.md
  • Moved one anchor comment in docker-compose/examples/aws-rds-databases/docker-compose.override.yaml above its key, so prettier does not glue it to the anchor

docker-compose/docker-compose.yaml was already clean and is untouched.

Test plan

  • Every changed yaml/json file parses to the same tree before and after (yq -o=json diff)
  • .buildkite/prettier-check.sh passes under CI's pins from .tool-versions (node 16.7.0, yarn 1.22.4)
  • CI: validate-docker-compose and prettier-check

marcleblanc2 and others added 2 commits October 14, 2026 18:59
- @sourcegraph/prettierrc 3.0.2 -> 3.0.3
- Delete .prettierignore: .github/ and *.md are formatted now
- Run prettier on . instead of a js/md glob, so yaml and json are
  formatted too

Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-01a13d0e-07fe-73e0-9435-95a24da163ce
All yaml and json parse to the same tree before and after. Moved one
anchor comment in the aws-rds-databases example above its key so
prettier does not attach it to the anchor.

Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-01a13d0e-07fe-73e0-9435-95a24da163ce
@github-actions

Copy link
Copy Markdown

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