-
Notifications
You must be signed in to change notification settings - Fork 23
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
Documentation updates: contributor's guide, notebook name standardization, hiding code cells #147
Conversation
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.
This is a huge update for the documentation, and it looks great! A few small tweaks to think about, but overall 👍
Thanks for these suggestions, @mnlevy1981 . FYI I also re-built and fixed a few links, and it is building properly. The documentation still looks the same as the above screenshot. |
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.
LGTM!
This PR will address issues raised in #146, #96, and #60. The key items it addresses are as follows:
Still need to do:
key_metrics
examples that need hidingpre-commit
check)?