Skip to content

chore: Upgrade prettier to 3.9.10 and format all files - #1441

Closed
marcleblanc2 wants to merge 2 commits into
rewrite-readme-docker-compose-onlyfrom
upgrade-prettier
Closed

marcleblanc2 wants to merge 2 commits into
rewrite-readme-docker-compose-onlyfrom
upgrade-prettier

Conversation

@marcleblanc2

@marcleblanc2 marcleblanc2 commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

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

@github-actions

github-actions Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

@marcleblanc2
marcleblanc2 changed the base branch from remove-single-container-leftovers to rewrite-readme-docker-compose-only October 30, 2026 01:46
@marcleblanc2
marcleblanc2 added this pull request to stack #1458 October 30, 2026 03:03
@marcleblanc2 marcleblanc2 changed the title chore: Upgrade prettier to 3.9.10 and format every file type chore: Upgrade prettier to 3.9.10 and format all files Oct 30, 2026
@marcleblanc2
marcleblanc2 requested a review from a team October 30, 2026 03:47
@marcleblanc2
marcleblanc2 enabled stacked auto-merge (squash) November 9, 2026 23:36
stacked auto-merge was automatically disabled November 10, 2026 17:54

Pull Request is not mergeable

marcleblanc2 and others added 2 commits November 10, 2026 12:43
- @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
@marcleblanc2
marcleblanc2 enabled stacked auto-merge (squash) November 10, 2026 19:50
@michaellzc

Copy link
Copy Markdown
Member

I would like us to revisit in the future. it's something we should keep consistent with the monorepo. tho not something we will have time to do anytime soon.

@michaellzc michaellzc closed this Nov 10, 2026
stacked auto-merge was automatically disabled November 10, 2026 20:18

Pull request was closed

An error occurred while trying to automatically change base from rewrite-readme-docker-compose-only to main November 10, 2026 20:48
@marcleblanc2
marcleblanc2 deleted the upgrade-prettier branch November 11, 2026 23:38
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