Skip to content

chore(tooling): add mdx to prettier and cspell, ignore archived docs - #4624

Open
thetaPC wants to merge 1 commit into
mainfrom
FW-6456-pt1
Open

chore(tooling): add mdx to prettier and cspell, ignore archived docs#4624
thetaPC wants to merge 1 commit into
mainfrom
FW-6456-pt1

Conversation

@thetaPC

@thetaPC thetaPC commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Issue URL: internal

What is the current behavior?

Prettier and cspell both only look at .md, so the tracked .mdx files in the repo have never been formatted or spellchecked. docs/deployment/play-store.mdx and its v7 copy have drifted as a result.

Separately, both tools run over versioned_docs/version-v5 and versioned_docs/version-v6. Those versions are not listed in versions.json, so Docusaurus never builds them. They are served from standalone deployments listed in versionsArchived.json, which means tooling is formatting and spellchecking 696 files that ship from somewhere else.

What is the new behavior?

.mdx is treated as a first class doc extension by both tools, and archived versions are excluded from both.

  • prettier glob now includes mdx
  • spellcheck glob now includes mdx
  • .prettierignore and cspell.json both skip versioned_docs/version-v5 and versioned_docs/version-v6
  • Formatted the two live .mdx files, which are now in scope

Does this introduce a breaking change?

  • Yes
  • No

Other information

This is groundwork for migrating our docs from .md to .mdx, which Docusaurus recommends ahead of v4.

@vercel

vercel Bot commented Aug 7, 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 7, 2026 11:17pm

Request Review

@thetaPC
thetaPC marked this pull request as ready for review August 7, 2026 23:21
@thetaPC
thetaPC requested a review from a team as a code owner August 7, 2026 23:21
@thetaPC
thetaPC requested a review from OS-jacobbell August 7, 2026 23:21
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