-
Notifications
You must be signed in to change notification settings - Fork 168
Add open binder and download link pre-amble (remove zip tooling) #2293
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
Add open binder and download link pre-amble (remove zip tooling) #2293
Conversation
for more information, see https://pre-commit.ci
|
Thanks for this work, @nilodna. Nice! Do you know if it's possible to add a bit larger vspace between the binder badge and the title of the Notebook? They're very close together now (see screenshot below) |
|
The code is a HTML snippet, so it should be easy @erikvansebille. I can do this today during the presentations. |
|
Did a quick check, and it looks like we don't want READTHEDOCS_VERSION but we want READTHEDOCS_GIT_IDENTIFIER. updating now :) |
VeckoTheGecko
left a comment
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.
…cels-code#2293) Co-authored-by: Nick Hodgskin <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…) (#2316) Co-authored-by: Danilo A. Silva <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>


Add at the top of all jupyter notebook examples page a button to binder or a direct link to the source code fixes #2206
EDIT VeckoTheGecko: This PR also removes the tooling for zipping up notebook examples, which is no longer needed.