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.

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

fix section header out of sequence warnings: #34

Merged
merged 1 commit into from
Mar 22, 2024
Merged

fix section header out of sequence warnings: #34

merged 1 commit into from
Mar 22, 2024

Conversation

kbroch-rivosinc
Copy link
Contributor 

asciidoctor: WARNING: riscv-zabha.adoc: line 53: section title out of sequence: expected levels 0 or 1, got level 2
asciidoctor: WARNING: riscv-zabha.adoc: line 63: section title out of sequence: expected levels 0 or 1, got level 2

@ved-rivos
Copy link
Collaborator

@kbroch-rivosinc - I have this intentionally since I want the "Preamble" "Copyright and license information" and "Contributors" all on same page.

@kbroch-rivosinc
fix section header out of sequence warnings:
21c93eb 
```
asciidoctor: WARNING: riscv-zabha.adoc: line 53: section title out of sequence: expected levels 0 or 1, got level 2
asciidoctor: WARNING: riscv-zabha.adoc: line 63: section title out of sequence: expected levels 0 or 1, got level 2
```

Signed-off-by: Kevin Broch <[email protected]>
@kbroch-rivosinc
Copy link
Contributor Author

kbroch-rivosinc commented Mar 22, 2024 •
edited
Loading

@kbroch-rivosinc - I have this intentionally since I want the "Preamble" "Copyright and license information" and "Contributors" all on same page.

That makes sense. Take a look at this commit again. I put a "Preamble" H1 header directly in, which allows those H2's to fall under it and end up on same page as before. This also has the benefit of having the html version have the same "Preamble" header which wasn't the case before.

@ved-rivos
Copy link
Collaborator

@kbroch-rivosinc Thanks! Looks good. Merging.

@ved-rivos ved-rivos merged commit 8f5242e into riscv:main Mar 22, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kbroch-rivosinc @ved-rivos