pester / docs

Source files for the Pester website.
https://pester.dev
MIT License
41 stars 77 forks source link

Docusaurus v3 #272

Closed fflaten closed 1 year ago

fflaten commented 1 year ago

Upgrades the docs website to Docusaurus v3 with MDX v3, prism-react-renderer v2 and React 18. Also includes some minor dependency-cleanup.

Related links:

Lessons learned during migration:

Feel free to commit to this with new builds or other fixes. Just thought we could verify early.

Fix #273

bravo-kernel commented 1 year ago

Thanks for creating this one. We still have some time but this makes it pretty clear what needs to be done. πŸ‘

bravo-kernel commented 1 year ago

And yes, the Alt3 module should definitely be able escape the curly brackets (especially since that fix is compatible with docusaurus v2)

bravo-kernel commented 1 year ago

@fflaten would you be able to test the Alt3 module artifact produced in https://github.com/alt3/Docusaurus.Powershell/actions/runs/5829982515?pr=169?

If it generates the expected markdown with escaped curly brackets, I will publish a new version.

fflaten commented 1 year ago

No diff besides comment in sidebar. All good! πŸŽ‰

bravo-kernel commented 1 year ago

Thank you for testing, much appreciated. If you find anything else feel free to let me know πŸ’ƒ

Released in https://www.powershellgallery.com/packages/Alt3.Docusaurus.Powershell/1.0.35

fflaten commented 1 year ago

Great work! Will do. Hopefully not more issues to come.

fflaten commented 1 year ago

3.0-rc.0 adds support for TS config files. Switch or stay with JS? https://github.com/facebook/docusaurus/pull/9317

bravo-kernel commented 1 year ago

I have been waiting to see how they do the ts config and would (and will) personally go for it.

fflaten commented 1 year ago

I have been waiting to see how they do the ts config and would (and will) personally go for it.

Cool. I'll let you handle that in a separate PR πŸ™‚

fflaten commented 1 year ago

Not sure if we'd like to merge this (when reviewed) or not. Feel free to close if it should be split into multiple PRs :)

bravo-kernel commented 1 year ago

I have no issues with merging this as it is.

I could not detect any issues in the preview so just let me know if you feel confident as well and I will merge.

fflaten commented 1 year ago

Nice. LGTM on my end πŸ™‚

bravo-kernel commented 1 year ago

Woohoo πŸ’ƒ