chore: fix prettier - #994
Conversation
PR SummaryLow Risk Overview In In Reviewed by Cursor Bugbot for commit df46e25. Bugbot is set up for automated code reviews on this repo. Configure here. |
There was a problem hiding this comment.
Pull request overview
This PR applies Prettier-driven formatting fixes to the Cloudflare Worker configuration and the generated/checked-in TypeScript declaration file to align with consistent code style.
Changes:
- Add a trailing comma / normalize formatting in
wrangler.jsoncfor theroutesarray entry. - Reformat TypeScript declarations in
worker-configuration.d.ts(e.g., unquoted method name and condensed union types).
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| wrangler.jsonc | Adjusts JSONC formatting (adds trailing comma) to satisfy Prettier/style expectations. |
| worker-configuration.d.ts | Applies Prettier formatting to declaration syntax (method name and union/mapped type formatting). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
This file is auto-generated, I don't think it should be formatted? I'm not sure why it is even checked in?
There was a problem hiding this comment.
The "official" recommendation is to check it in, which is why it's here, but I'm more in favor of just removing it now and adding a step in the contributing guideline saying to generate it yourself
There was a problem hiding this comment.
Can we just add it as a prepare script, so it happens whenever deps get installed locally?
As titled