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

Fix manual #834

Merged
merged 5 commits into from Jan 2, 2020
Merged

Fix manual #834

merged 5 commits into from Jan 2, 2020

Conversation

akrabat
Copy link
Member

@akrabat akrabat commented Jan 2, 2020

The title levels in manual.rst are inconsistent and this is a fatal error in docutils.

In addition, DEVELOPERS.rst is pulled into manual.rst, so the title levels in DEVELOPERS.rst need to be adjusted so that the sections within DEVELOPERS.rst sit within the chapter in manual.

Finally, the comment text in the extensions are also pulled into manual.rst, so the dotted_toc comment needed updating so that it didn't create a new chapter in the manual.

For reference when reviewing, the levels used by the manual are:

level 1: ============
level 2: ------------
level 3: ~~~~~~~~~~~~
level 4: ************
level 5: """"""""""""

Fixes #833

Ensure that the same marks are used for the same title level throughout
the manual as otherwise it will not build with newer docutils.

The level underlines in use are:

    level 1: ============
    level 2: ------------
    level 3: ~~~~~~~~~~~~
    level 4: ************
    level 5: """"""""""""
The embedded docs are included in the manual and so we need to ensure
that we do not use a title level here as it will create a new section.
Hence, we remove the title for "history" and also fix the bullet points
so that the are valid rST.
DEVELOPERS.rst is included in the manual and so it needs to use the same
title levels as manual.rst such that it embeds into the manual below the
chapter level.
It looks better if the Guidelines heading is included.
The author meta tag and the license now list as "Roberto Alsina and
the contributors to the rst2pdf project".
@akrabat akrabat added this to the 0.96 milestone Jan 2, 2020
@akrabat akrabat requested a review from lornajane January 2, 2020 18:10
Copy link
Contributor

@lornajane lornajane left a comment

Choose a reason for hiding this comment

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

Yes, this makes the manual build on my local Python 3.7 install. Thanks!

@lornajane lornajane merged commit 9087b80 into rst2pdf:master Jan 2, 2020
@akrabat akrabat deleted the fix-manual branch January 2, 2020 22:21
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.

Incorrect processing of headings.
2 participants