Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Removed our custom ID stripping since upstream deals with them now #232

Merged
merged 2 commits into from
Dec 21, 2018

Conversation

obilodeau
Copy link
Member

Fixes #192 but also removes our workaround in #150.

Test cases from #192 were added.

Will impact links because the prepending of _ is reintroduced on automatically generated links (Asciidoctor's behavior) so CHANGELOG will be explicit about that.

Fixes asciidoctor#192 but also removes our workaround in asciidoctor#150.

Test cases from asciidoctor#192 were added.

Will impact links because the prepending of `_` is reintroduced on automatically generated links (Asciidoctor's behavior) so CHANGELOG will be explicit about that.
@obilodeau obilodeau mentioned this pull request Dec 12, 2018
@obilodeau obilodeau added this to the 2.0.0 milestone Dec 12, 2018
@obilodeau obilodeau merged commit adbad9a into asciidoctor:master Dec 21, 2018
@obilodeau obilodeau deleted the id-stripping-no-longer-needed branch February 11, 2019 04:52
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant