Skip to content

fix: correct new line chars in mdx#31789

Merged
colbywhite merged 1 commit into
productionfrom
mdx-whitespace-cleanup
Jun 30, 2026
Merged

fix: correct new line chars in mdx#31789
colbywhite merged 1 commit into
productionfrom
mdx-whitespace-cleanup

Conversation

@colbywhite

Copy link
Copy Markdown
Collaborator

Replace blank lines that contained a single trailing space with truly empty lines. Pure whitespace cleanup with no rendered output change; makes the files cleaner under any MDX parser, Satteri or otherwise.

Replace blank lines that contained a single trailing space with truly
empty lines. Pure whitespace cleanup with no rendered output change;
makes the files cleaner under any MDX parser.
@github-actions

Copy link
Copy Markdown
Contributor

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

Pattern Owners
/src/content/changelog/ @cloudflare/pm-changelogs, @cloudflare/product-owners
/src/content/changelog/security-center/ @jwcrisp, @alexmoraru7, @cloudflare/appsec-reviewers, @elithrar, @danielegm, @cloudflare/pm-changelogs, @cloudflare/product-owners, @davejbax, @zrkn, @hemanthk1099, @bseel-cfone
/src/content/docs/api-shield/ @patriciasantaana, @cloudflare/appsec-reviewers, @elithrar, @xmflsct, @danielegm, @cloudflare/product-owners
/src/content/docs/cloudflare-one/ @ranbel, @Maddy-Cloudflare, @codyanthony850, @cloudflare/cf1-reviewers, @elithrar, @cloudflare/product-owners
/src/content/docs/d1/ @elithrar, @rita3ko, @irvinebroque, @vy-ton, @ivoryibu, @rts-rob, @joshthoward, @lambrospetrou, @oxyjun, @cloudflare/product-owners
/src/content/docs/ddos-protection/ @patriciasantaana, @cloudflare/appsec-reviewers, @elithrar, @alpdot, @cloudflare/product-owners, @anita-tenjarla, @cjdoucette, @sbohrer, @vcfxb, @danielegm
/src/content/docs/learning-paths/ @cloudflare/product-owners
/src/content/docs/log-explorer/ @angelampcosta, @dcpena, @cloudflare/product-owners
/src/content/docs/logs/ @soheiokamoto, @angelampcosta, @rianvdm, @dcpena, @cloudflare/product-owners
/src/content/docs/speed/ @cloudflare/product-owners, @ack-cf
/src/content/docs/waiting-room/ @angelampcosta, @dcpena, @cloudflare/product-owners, @hsaxenaCF, @danielegm
* @cloudflare/product-owners

@ask-bonk ask-bonk Bot added documentation Documentation edits quick win size/xs labels Jun 29, 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.

Review Summary

This is a clean, mechanical whitespace-only change that removes trailing spaces from blank lines across 24 MDX files.

What changes:

  • Replaces space-only blank lines with truly empty blank lines
  • Affects changelog entries, docs pages, and partials across multiple products
  • No rendered output change expected

Checks performed:

  • Frontmatter: Not modified (safe)
  • Links: Not touched
  • Style guide: No violations introduced; this change actually improves conformance
  • MDX parsing: Space-only lines and empty lines behave identically in MDX, so this is structurally safe
  • Size: XS (mechanical cleanup)

Labels applied: , ,

Non-blocking observation:

  • is missing a trailing newline at EOF (pre-existing, not introduced by this PR). Consider fixing in a follow-up or adding it here since the file is already being touched.

@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.

Review Summary

This is a clean, mechanical whitespace-only change that removes trailing spaces from blank lines across 24 MDX files.

What changes:

  • Replaces space-only blank lines with truly empty blank lines
  • Affects changelog entries, docs pages, and partials across multiple products
  • No rendered output change expected

Checks performed:

  • Frontmatter: Not modified (safe)
  • Links: Not touched
  • Style guide: No violations introduced; this change actually improves conformance
  • MDX parsing: Space-only lines and empty lines behave identically in MDX, so this is structurally safe
  • Size: XS (mechanical cleanup)

Labels applied: documentation, quick win, size/xs

Non-blocking observation:

  • src/content/changelog/workers-ai/2025-10-02-deepgram-flux.mdx is missing a trailing newline at EOF (pre-existing, not introduced by this PR). Consider fixing in a follow-up or adding it here since the file is already being touched.

@ask-bonk

ask-bonk Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Done.

PR #31789 reviewed and approved.

Summary:

  • Type: Mechanical whitespace cleanup across 24 MDX files
  • Risk: Very low — replaces space-only blank lines with empty blank lines; no rendered output change
  • Issues found: None blocking. Pre-existing missing trailing newline noted in 2025-10-02-deepgram-flux.mdx
  • Labels applied: documentation, quick win, size/xs

The change is safe to merge.

github run

@xmflsct xmflsct removed their assignment Jun 29, 2026
@cloudflare-docs-bot

cloudflare-docs-bot Bot commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Review

✅ No issues found in commit 949eed1.

Code Review

This code review is in beta and may not always be helpful — use your judgment.

No code review issues found.

Conventions

Checks PR title, description, and redirect checklist.

No convention issues found.

Style Guide Review

No style-guide issues found.

Redirects

No missing redirect entries found.

Commands

Only codeowners can run commands. Post a comment with the command to trigger it.

Command Description
/review Runs a review now. Incremental if a prior review exists, full if not.
/full-review Re-reviews the entire PR diff from scratch, ignoring incremental history. Useful after a rebase, when you want a fresh review, or if the bot gets out of sync and reports issues that no longer exist.
/ignore-review-limit Permanently lifts the 2-review automatic limit for this PR. Future pushes will trigger reviews as normal.
/disable-auto-review Stops automatic reviews from triggering on future pushes to this PR. Codeowners can still run /review or /full-review manually.

@colbywhite colbywhite enabled auto-merge (squash) June 30, 2026 18:48
@colbywhite colbywhite merged commit f302a0a into production Jun 30, 2026
18 checks passed
@colbywhite colbywhite deleted the mdx-whitespace-cleanup branch June 30, 2026 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.