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

Textile reader: Add support for link references #8706

Merged
merged 2 commits into from Jun 22, 2023
Merged

Conversation

alts
Copy link
Contributor

@alts alts commented Mar 19, 2023

Textile supports what it calls a "link alias", which is analogous to Markdown's reference-style links.

During the parser's first-pass, we can collect link references in addition to footnotes. Textile differs from Markdown in that reference definitions don't have titles nor attributes, so the stored values in stateKeys is relatively sparse.

I modeled the parser after the Markdown, and the new test cases are passing, but let me know if there's anything essential from Markdown's reference parsing that I missed.

Textile supports what it calls "link alias", which are analogous to
Markdown's reference-style links.

During the parser's first-pass, we can collect link references in
addition to footnotes. Textile differs from Markdown in that
reference definitions don't have titles nor attributes, so the
stored values in `stateKeys` is relatively sparse.
@alts alts changed the title Add support for link references in Textile Textile reader: Add support for link references Mar 23, 2023
@jgm jgm merged commit 83b69ea into jgm:main Jun 22, 2023
10 checks passed
@jgm
Copy link
Owner

jgm commented Jun 22, 2023

thanks and sorry for the belated merge!

@alts
Copy link
Contributor Author

alts commented Jun 22, 2023

@jgm No worries! Thank you so much for the work you do maintaining this!

If there's anything I can do to make the PRs easier to review or merge, please let me know. I anticipate making a few more upgrades to the Textile parser as I bump into discrepancies while I port a project of mine.

@alts alts deleted the alts/link-refs branch June 22, 2023 18:07
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.

None yet

2 participants