Skip to content

chore(prettier): ignore ejected swizzles, not wrapped ones - #4617

Open
thetaPC wants to merge 1 commit into
mainfrom
swizzled-prettier
Open

chore(prettier): ignore ejected swizzles, not wrapped ones#4617
thetaPC wants to merge 1 commit into
mainfrom
swizzled-prettier

Conversation

@thetaPC

@thetaPC thetaPC commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Issue URL:

What is the current behavior?

src/theme/README.md says swizzled components should be added to the prettier ignore so they stay diffable against @docusaurus/theme-classic, but only DocItem and DocRoot were listed.

What is the new behavior?

  • .prettierignore lists the seven ejected components. Wrapped components (using @theme-original) are our own code, so they keep repo formatting: DocSidebar, TOC, MDXComponents/index.tsx, and DocRoot/Layout/Main.
  • The seven ejected files are reformatted to upstream's prettier settings.

Does this introduce a breaking change?

  • Yes
  • No

Other information

N/A

@vercel

vercel Bot commented Aug 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ionic-docs Ready Ready Preview Aug 5, 2026 9:36pm

Request Review

@thetaPC
thetaPC marked this pull request as ready for review August 5, 2026 21:53
@thetaPC
thetaPC requested a review from a team as a code owner August 5, 2026 21:53
@thetaPC
thetaPC requested review from OS-jacobbell and ShaneK and removed request for OS-jacobbell August 5, 2026 21:53
@thetaPC thetaPC mentioned this pull request Aug 5, 2026
2 tasks
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