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): remove tags hiding end of the content #53

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

GitFenixZ
Copy link

@GitFenixZ GitFenixZ commented Apr 14, 2024

I removed the details and summary HTML tags used to wrap ./test/negative-tests/for-mdx/mdx_cases.ml.corrected.expected's part=matching that were hiding the end of README.mdx and breaking the code block.

This is how README.mdx looks without this PR.
image
image

To see how it looks with this PR:
https://github.com/GitFenixZ/ppx_expect/tree/fix-readme-hidden-content

PS: I don't know much about MDX, so I'm not sure if this is even a "fix" or the best way to fix it. If you have a better solution, go for it!

I removed the `details` and `summary` HTML tags used to wrap
`./test/negative-tests/for-mdx/mdx_cases.ml.corrected.expected`'s
`part=matching` that were hiding the end of `README.mdx`.

PS: I don't know much about MDX, so I'm not sure if this is even a "fix"
or the best way to fix it. If you have a better solution, go for it!

Signed-off-by: GitFenixZ <[email protected]>
@GitFenixZ GitFenixZ force-pushed the fix-readme-hidden-content branch from b19b7b9 to b2205a6 Compare April 14, 2024 00:23
@github-iron github-iron added the forwarded-to-js-devs This report has been forwarded to Jane Street's internal review system. label Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
forwarded-to-js-devs This report has been forwarded to Jane Street's internal review system.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants