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: correctly consume the child block, with example fenced code block #13

Merged

Conversation

dkarlovi
Copy link
Contributor

@dkarlovi dkarlovi commented Jan 9, 2024

This PR highlights the issue with the extensions where it will eat all the spaces from a nested code block. I'm not sure if it's due to the extension or a bug upstream.

@dkarlovi
Copy link
Contributor Author

dkarlovi commented Jan 9, 2024

See thephpleague/commonmark#1008, this now works as expected, ready for review.

@dkarlovi dkarlovi changed the title tests: add a failing test for code block fix: correctly consume the child block, with example fenced code block Jan 9, 2024
Copy link
Owner

@JanStanleyWatt JanStanleyWatt left a comment

Choose a reason for hiding this comment

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

I Confirmed that the test passes.

@JanStanleyWatt JanStanleyWatt merged commit d3406b3 into JanStanleyWatt:master Jan 10, 2024
@JanStanleyWatt
Copy link
Owner

Thanks all for your contrebute!

@dkarlovi dkarlovi deleted the fix/code-block-newlines branch January 10, 2024 12:18
@dkarlovi
Copy link
Contributor Author

Thank you for the review and working on this project @JanStanleyWatt, looking forward to seeing this bugfix released!

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