Skip to content

Fix SetexToAtx #607

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Nov 24, 2022
Merged

Fix SetexToAtx #607

merged 3 commits into from
Nov 24, 2022

Conversation

@lggruspe
Copy link
Contributor

@lggruspe lggruspe commented Oct 25, 2022

Changes:

  • Fix range in substitution of level 2 headings in SetexToAtx.
  • Tests

Fixes #606

alerque
alerque approved these changes Nov 24, 2022
View reviewed changes
Copy link
Member

@alerque alerque left a comment

Choose a reason for hiding this comment

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

Thanks for taking the time to contribute, and especially for both explaining the issue and including tests to go with the change!

@alerque alerque merged commit 2cb8d9d into preservim:master Nov 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alerque alerque alerque approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

SetexToAtx sometimes ignores level 2 headings

2 participants

@lggruspe @alerque