Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade docs to Docusaurus 3 #261

Merged
merged 6 commits into from Jan 11, 2024
Merged

Upgrade docs to Docusaurus 3 #261

merged 6 commits into from Jan 11, 2024

Conversation

josh-collinsworth
Copy link
Contributor

@josh-collinsworth josh-collinsworth commented Jan 8, 2024

NOTE: for some reason, this version of MDX (or something) has a few issues with certain characters in Markdown files (< and >), so there were a handful of places I had to swap those out with HTML entities.

@josh-collinsworth josh-collinsworth changed the title Joco/upgrade to docusaurus 3 Upgrade docs to Docusaurus 3 Jan 8, 2024
Comment on lines +152 to +153
| Meta &lt; | Move to first entry in history |
| Meta &gt; | Move to last entry in history |
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had to remove the - here, because -> creates an arrow, and that would be confusing.

Related: should these all have + characters instead of - characters? Or is it not a combined keyboard shortcut like that?

Comment on lines +274 to +276
[Helix](https://helix-editor.com) comes with built-in language server support.
Enabling connection to the Deno language server requires changes in the
`languages.toml` configuration file.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is just auto-formatting.

Copy link
Member

@ry ry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but I think @kwhinnery should review

@donjo
Copy link
Member

donjo commented Jan 10, 2024

Functionally all seems to be working on my end as well

@kwhinnery kwhinnery merged commit 2ce701f into main Jan 11, 2024
3 checks passed
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.

None yet

4 participants