Skip to content

chore: run format#31336

Merged
MohamedH1998 merged 3 commits into
productionfrom
chore/run-format
Jun 9, 2026
Merged

chore: run format#31336
MohamedH1998 merged 3 commits into
productionfrom
chore/run-format

Conversation

@MohamedH1998

Copy link
Copy Markdown
Contributor

Summary

Run the pnpm format command on the repo and fix any errors thrown

Screenshots (optional)

Documentation checklist

  • Is there a changelog entry (guidelines)? If you don't add one for something awesome and new (however small) — how will our customers find out? Changelogs are automatically posted to RSS feeds, the Discord, and X.
  • The change adheres to the documentation style guide.
  • If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
  • Files which have changed name or location have been allocated redirects.

@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
package.json @cloudflare/content-engineering
* @cloudflare/product-owners
*.astro @cloudflare/content-engineering, @kodster28
/src/content/catalog-models/ @abhishekkankani, @palashgo, @thebongy, @roerohan, @kathayl, @mchenco, @zeke, @superhighfives, @bfirsh, @mattrothenberg, @ethulia, @cloudflare/content-engineering, @cloudflare/product-owners
/src/content/docs/dns/ @RebeccaTamachiro, @hannes-cf, @cloudflare/product-owners, @esomoza-cf, @fattouche, @xofyarg, @dklbreitling, @chreo, @svenr-cf, @kerolasa, @matildeopbravo, @vavrusa, @mworsley-cloudflare, @sebastiaanyn, @vendemiat, @Woutifier
/src/content/release-notes/beacon-min-js.yaml @angelampcosta, @dcpena, @ryantownsend, @cloudflare/product-owners

@ask-bonk ask-bonk Bot added the engineering Problems or updates to developers.cloudflare.com website label Jun 9, 2026

@ask-bonk ask-bonk Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pure formatting run with no issues found.

What changed:

  • Whitespace, indentation, and line-wrapping fixes across 100 files (Astro components, Markdown, JSON, YAML)
  • Trailing newlines added to files missing them
  • Markdown tables reformatted in RealtimeKit API reference docs
  • .prettierignore updated to exclude src/content/agent-setup/prompt.md (appropriate, since that file is consumed raw by agents and should not be auto-formatted)

No semantic changes, broken MDX syntax, or frontmatter issues detected. Safe to merge.

@ask-bonk

ask-bonk Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

I reviewed PR #31336 and found it to be a clean, mechanical formatting run.

Actions taken:

Summary of changes:
The PR runs pnpm format across 100 files, resulting in ~3,800 insertions and ~5,000 deletions that are purely stylistic:

  • Astro components: quote normalization, attribute wrapping, JSX expression indentation
  • Markdown docs (primarily RealtimeKit API references): table alignment, list indentation, trailing newlines
  • JSON data files (src/content/catalog-models/, worker/tsconfig.json): standard Prettier formatting
  • YAML files (Stream configs, glossary, release notes): indentation and whitespace fixes
  • package.json / pnpm-workspace.yaml: stray blank lines removed
  • .prettierignore: added src/content/agent-setup/prompt.md so the raw agent prompt file stays unformatted

Issues found: None. No MDX syntax was broken, no semantic changes were introduced, and no frontmatter was altered. The cloudflare-docs bot also confirmed no style-guide violations in the latest commit. This PR is safe to merge.

github run

@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

@MohamedH1998

Copy link
Copy Markdown
Contributor Author

/bonk review

@ask-bonk

ask-bonk Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Review for PR #31336: chore: run format

I spot-checked the diff and this PR is indeed a purely mechanical formatting pass.

  • Astro components — clean indentation, attribute wrapping, and quote normalization with no logic changes.
  • JSON catalog models — consistent Prettier output; all data fields and schemas remain intact.
  • YAML / workspace config — whitespace and alignment fixes only.
  • package.json — formatting is limited to indentation; no dependency bumps or script alterations.

The .prettierignore update to exclude src/content/agent-setup/prompt.md is the correct call, since that file is consumed raw by agents and should stay unformatted.

No semantic changes, broken MDX syntax, or data loss detected. This is a safe, cosmetic cleanup.

Verdict: Approve. No issues found — safe to merge.

github run

@MohamedH1998 MohamedH1998 merged commit cd305a1 into production Jun 9, 2026
13 checks passed
@MohamedH1998 MohamedH1998 deleted the chore/run-format branch June 9, 2026 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

engineering Problems or updates to developers.cloudflare.com website product:changelog product:logs Related to Logs product:realtime product:support product:workers Related to Workers product size/xl

Projects

None yet

Development

Successfully merging this pull request may close these issues.