Skip to content

Conversation

@arijitde92
Copy link
Contributor

Made changes in the .readthedocs.yaml to enable format for downloading pdf and epub versions of the documentation.
Also made changes in the docs/conf.py to make the generated pdfs without blank pages.

Implemented same fix as done in this issue which was merged in this PR.

I have added pdf and epub options in the .readthedocs.yaml file and have tested the documentation in readthedocs. The pdf and epub versions of the documentation can now be downloaded.

You can check the generated RTD pdf from my forked repository here

@AyanSinhaMahapatra please check and let me know if any other changes are required in this PR.

Tasks

  • PR is descriptively titled 📑 and links the original issue above 🔗
  • Tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR
    Run tests locally to check for errors.
  • Commits are in uniquely-named feature branch and has no merge conflicts 📁

Signed-off-by: Arijit De [email protected]

Copy link
Member

@AyanSinhaMahapatra AyanSinhaMahapatra left a comment

Choose a reason for hiding this comment

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

Thanks, see comments above.

@arijitde92
Copy link
Contributor Author

Hi @AyanSinhaMahapatra , I squashed the commit into one commit. Please check and let me know if this is okay.

@AyanSinhaMahapatra
Copy link
Member

@arijitde92 looks good, thanks++
Just a small nit again :P
It would be much nicer if you can amend the commit and have a more concise and to-the-point commit message. Since this is a very small change so the history and all the commit details is not relevant here. Just a regular commit title, body and signoff. The DCO fails because of the formatting, it will fix that also.

Made changes in the .readthedocs.yaml to enable format for downloading pdf and epub versions of the documentation and added latex_elements in the conf.py file which generates the pdf without blank pages. The minimum version requirement for sphinx was 6.2.1 which was causing build failure in read the docs, hence changing it 3.3.1 as written in setup.cfg of nexB/aboutcode

Signed-off-by: Arijit De <[email protected]>
@arijitde92
Copy link
Contributor Author

I amended it. I hope this is enough for the merge. If anything else is required, please let me know.

Copy link
Member

@AyanSinhaMahapatra AyanSinhaMahapatra left a comment

Choose a reason for hiding this comment

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

Thanks! Merging!

@AyanSinhaMahapatra AyanSinhaMahapatra merged commit 5072e43 into aboutcode-org:main Jun 1, 2023
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.

2 participants