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

fix: remove md that is incompatible with docs #1063

Merged
merged 1 commit into from
Jun 28, 2023

Conversation

oddgrd
Copy link
Contributor

@oddgrd oddgrd commented Jun 28, 2023

Description of change

This fixes the parsing error in the docs.

How has this been tested? (if applicable)

Tested with yarn mintlify dev in the docs repo on latest mintlify.

Comment on lines -403 to +401
[^1]: <https://git-scm.com/book/en/v2/Customizing-Git-Git-Configuration#_core_autocrlf>
[^1]: https://git-scm.com/book/en/v2/Customizing-Git-Git-Configuration#_core_autocrlf
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jonaro00 do you recall why you changed this? Removing it fixes the parsing error in the docs, but it might be needed?

Copy link
Member

@jonaro00 jonaro00 Jun 28, 2023

Choose a reason for hiding this comment

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

Mainly due to markdown lint rules, but as you also have discovered, Mintlify's markdown parser does not even support comments (to disable rules)...

Copy link
Member

Choose a reason for hiding this comment

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

In my other large docs PR, using Mintlify v3, I changed the Contributing sidebar item to be a link to CONTRIBUTING.md instead. Sacrificing the quality (graph etc) from GitHub's rendering to mirror it is a waste IMO. A contributor should not be afraid of starting out on a markdown file on GH.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, that is a very fair point, there certainly are some footguns, I encountered a few myself testing this. Our understanding was that they would support mermaid soon, but they must not have had time to integrate it yet. 😅

I will merge this for now so we fix the parsing error, but I think just linking to github is definitely worth considering. I'll ask Ivan if they replied about mermaid as well.

@oddgrd oddgrd merged commit 9ee11ed into shuttle-hq:main Jun 28, 2023
@oddgrd oddgrd deleted the fix/contribution-doc-invalid-mdx branch June 28, 2023 17:11
AlphaKeks pushed a commit to AlphaKeks/shuttle that referenced this pull request Jul 21, 2023
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.

2 participants