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

Added dependency chapter #2

Merged
merged 3 commits into from Jul 3, 2021
Merged

Conversation

DavideTisi
Copy link
Contributor

added a subsection on Dependencies. I have tested generating the pdf seems to work and it is not terribly ugly

added dependency
fixed table
tested on pdflatex building
@matteosecli
Copy link
Owner

Thanks @DavideTisi! 🙂

There is a warning on the first code block (guess it's best to leave an empty line), but I'm going to merge anyway and fix it later.

It's a fault on my side for not having provided instructions on how to locally build the documentation, and for the absence of a CI workflow that automatically builds the HTML docs for each PR, in order to check. I've now put some instructions in the README and I've also set up a CI workflow, so that the next contributions will be automatically built and made available as downloads for inspection.

@matteosecli matteosecli merged commit 713e72d into matteosecli:master Jul 3, 2021
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.

None yet

2 participants