-
Notifications
You must be signed in to change notification settings - Fork 9
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
Better Package Style layout #71
Conversation
This is way better already. Agree that we don't need a separate page per topic. |
f5a6fc3
to
5e1d2c4
Compare
I decide to add Graphviz diagrams. These turned out to be really useful and more informative than the tree-based ones. In addition, taking advantage of the Notice these diagrams follow RST syntax and are rendered when executing Sphinx. |
This is great and maintainable! |
Graphviz is a little bit tricky when it comes to linking subgraphs. Nevertheless, I managed to solve this locally. The build-system diagram is going to look really cool! 🚀 |
About to add "api docs" and I'll need this. |
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
@jorgepiloto @akaszynski |
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
:caption: A Python library is a collection of packages. | ||
:alt: A Python library is a collection of packages. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think figure captions need to end with a period. Will modify the rest accordingly.
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
Thanks for such a deep review @PipKat! I applied all the suggested changes 👍🏽 |
Finally, merging this 🚀 |
Resolves #62 and partially addresses #64.