Skip to content

[CONTRIBUTING] Update instructions for how to build PDF files locally#319

Merged
stuij merged 1 commit intoARM-software:mainfrom
yury-khrustalev:patch/contrib
Apr 7, 2025
Merged

[CONTRIBUTING] Update instructions for how to build PDF files locally#319
stuij merged 1 commit intoARM-software:mainfrom
yury-khrustalev:patch/contrib

Conversation

@yury-khrustalev
Copy link
Copy Markdown
Contributor

Add instructions for how to build PDF files locally using Python virtual environment.
Also add a .gitignore file.

Copy link
Copy Markdown
Contributor

@smithp35 smithp35 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 adding the instructions. I've made one suggestion that might simplify the text, but its not that important if you prefer the original.

CONTRIBUTING.md Outdated
To check the outcome of your changes, use the `tools/rst2pdf/generate-pdfs.sh`
script. To install the (python) prerequisites for the script, do the following.

Create and activate Python virtual environment (you will need to use Python 3.x):
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We might be able to get rid of the (you will need to use Python 3.x) if we use python3 and pip3 explicitly.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I prefer to use a more generic commands like python in the code listings, but I'm happy to change it to python3 and remove notice about having to use Python 3.x to make it more concise.

Add instructions for how to build PDF files locally using
Python virtual environment. Also add a .gitignore file.
Copy link
Copy Markdown
Contributor

@smithp35 smithp35 left a comment

Choose a reason for hiding this comment

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

LGTM. I'll leave some time for others to comment. Give me a ping next week and I can merge it for you (apologies merge is somewhat locked down in this repo).

@stuij stuij merged commit 4fd6aec into ARM-software:main Apr 7, 2025
1 check passed
@yury-khrustalev yury-khrustalev deleted the patch/contrib branch February 20, 2026 11:47
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.

3 participants