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

Recursive chevrons are bananas #49

Merged
merged 4 commits into from
Sep 6, 2017

Conversation

kivikakk
Copy link

@kivikakk kivikakk commented Jul 13, 2017

Some janky testing which isn't supported on Windows, but it gets the job done.

This is to fix commonmark#218 — we trip a (different) regression test currently.

@Alhadis Alhadis mentioned this pull request Jul 16, 2017
@brentjo
Copy link
Member

brentjo commented Sep 5, 2017

@kivikakk Do you have a rough idea for when this PR will ship? Thanks again for all your work getting these issues patched up!

@kivikakk
Copy link
Author

kivikakk commented Sep 6, 2017

@brentjo This is unfortunately still pending a direction from upstream, hence commonmark#218 still being open. I might just merge this now to protect ourselves in the meantime; stand by.

@kivikakk kivikakk force-pushed the kivikakk/recursive-chevrons-are-bananas branch from 0272b8f to 37cd730 Compare September 6, 2017 00:43
@kivikakk kivikakk merged commit 820372d into master Sep 6, 2017
@kivikakk kivikakk deleted the kivikakk/recursive-chevrons-are-bananas branch September 6, 2017 01:12
@brentjo
Copy link
Member

brentjo commented Sep 6, 2017

🙇 thanks @kivikakk

martincizek added a commit to orchitech/cmark-gfm that referenced this pull request Apr 10, 2020
martincizek added a commit to orchitech/cmark-gfm that referenced this pull request May 4, 2021
martincizek added a commit to orchitech/cmark-gfm that referenced this pull request May 4, 2021
martincizek added a commit to orchitech/cmark-gfm that referenced this pull request May 4, 2021
talum pushed a commit that referenced this pull request Sep 14, 2021
* Add some pathological regression tests

* Fix new pathological case

* Cleanup.

* Give it a second, for Travis
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.

Pathological input: Unclosed inline links (another)
2 participants