Skip to content

[DOXIA-740] Add test for non-consecutive headings leading to incomplete#228

Merged
kwin merged 3 commits intomasterfrom
bugfix/markdown-non-consecutive-heading-levels
Sep 24, 2024
Merged

[DOXIA-740] Add test for non-consecutive headings leading to incomplete#228
kwin merged 3 commits intomasterfrom
bugfix/markdown-non-consecutive-heading-levels

Conversation

@kwin
Copy link
Copy Markdown
Member

@kwin kwin commented Sep 17, 2024

output

@kwin kwin force-pushed the bugfix/markdown-non-consecutive-heading-levels branch 2 times, most recently from 89e26e2 to cf6fbd9 Compare September 19, 2024 07:04
@kwin kwin marked this pull request as ready for review September 19, 2024 07:05
@kwin kwin requested a review from michael-o September 19, 2024 07:06
@kwin kwin force-pushed the bugfix/markdown-non-consecutive-heading-levels branch from cf6fbd9 to 8a70469 Compare September 19, 2024 07:07
IndexingSink was always expecting a sectionTitle below each section
which doesn't hold.
Also close bufferingSink when the section is ended or a new one started
(without a preceding sectionTitle)

Skip irrelevant index entries (non section type or invalid id) in TOC
@kwin kwin force-pushed the bugfix/markdown-non-consecutive-heading-levels branch from 8a70469 to a940182 Compare September 19, 2024 07:09
Comment thread doxia-core/src/main/java/org/apache/maven/doxia/index/IndexingSink.java Outdated
@kwin kwin merged commit c93d142 into master Sep 24, 2024
@kwin kwin deleted the bugfix/markdown-non-consecutive-heading-levels branch September 24, 2024 07:49
@jira-importer
Copy link
Copy Markdown

Resolve #866

@slachiewicz slachiewicz added this to the 2.1.0 milestone Feb 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants