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: readme note #2204

Closed
wants to merge 2 commits into from
Closed

fix: readme note #2204

wants to merge 2 commits into from

Conversation

dtdang
Copy link
Contributor

@dtdang dtdang commented Aug 2, 2024

What I did

Note in README wasn't rendering properly.

fixes: #

How I did it

How to verify it

Checklist

  • All changes are completed
  • New test cases have been added
  • Documentation has been updated

If a plugin does not originate from the [ApeWorX GitHub Organization](https://github.com/ApeWorX?q=ape&type=all), you will get a warning about installing 3rd-party plugins.
Install 3rd party plugins at your own risk.
```
> \[!NOTE\]
Copy link
Member

Choose a reason for hiding this comment

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

Oh nice! Is this specific to Github markdown, or does it also work with sphinx-markdown?

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 one is specific for Github markdown. I don't think it works the same with sphinx-markdown.

Copy link
Member

Choose a reason for hiding this comment

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

Remember that this is the quickstart guide for our docs as well

Copy link
Contributor Author

Choose a reason for hiding this comment

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

image

This will render in the docs like this which I think is still a good way of doing this since it makes it more clear on github about the note and is still clear enough when rendered with sphinx.

Copy link
Member

Choose a reason for hiding this comment

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

Personally I love the note highlighting we have in the docs now, I would love to have it render in github better but I think the docs rendering takes precedence over github

@dtdang dtdang closed this Aug 5, 2024
@dtdang dtdang deleted the fix/readme_notes branch August 5, 2024 14:43
@fubuloubu
Copy link
Member

fubuloubu commented Aug 5, 2024

As a side note, I'm very in favor of this type of change with the addition of an extension or plugin that converts the Github alert syntax to the equivalent HTML or directive syntax for Sphinx-flavored markdown (which uses CommonMark) so the docs look right

As a reference
https://github.com/antfu/markdown-it-github-alerts

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