style: enable prettier for mdx files

[saihaj]

Not sure why we disabled prettier for .mdx files. I was working on some other PR and saw it wasn't working so decided to enable for everything.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
graphql-github-io	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Dec 23, 2024 4:02pm

[dimaMachina]

@benjie should we disable proseWrap for mdx?

graphql.github.io/.prettierrc

Line 15 in c4306f1

"proseWrap": "always",

[benjie]

Hold fire on this.

[benjie]

Okay I dug up the bug with proseWrap in MDX / bug report.

Yes, we must disable proseWrap in MDX if we enable prettier; generally I don't trust it with MDX files. I've taken to just un-ignoring each new MDX file I add - perhaps we should ignore certain problematic files but enable?

[benjie]

@saihaj is it possible to generate the static website before and after this change and assert that there are no differences in the generated output?

[saihaj]

@saihaj is it possible to generate the static website before and after this change and assert that there are no differences in the generated output?

yes. The deploy preview we see on this PR is a version we can compare that with prod. We can build this branch and then try to diff between source branch build