-
Notifications
You must be signed in to change notification settings - Fork 112
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
Repo2docker container tests #26
Conversation
where the default python is 3.7, but we have used 3.9 for the demo.
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
[some duplication here]
(has now specific workflow)
Nice work, @RobertRosca . I did complete this (to a functional level) hopefully in the way you had planned this. Do you want to look at it before I merge? |
Sure, I'll have a look now |
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.
Apart from the makefile/docker image name comments, LGTM (to both of us I guess 😂)
Co-authored-by: Robert Rosca <32569096+RobertRosca@users.noreply.github.com>
Co-authored-by: Robert Rosca <32569096+RobertRosca@users.noreply.github.com>
Adds another job to the staging workflow called
repo2docker-build-and-test
which builds a docker container with repo2docker usingjupyterhub/repo2docker-action
and then executes nbval tests inside that container.This checks that running the notebooks on MyBinder works correctly.