-
Notifications
You must be signed in to change notification settings - Fork 19
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
Periodically build notebooks to ensure they haven't broken #43
Comments
I would really like to look into this, can it be assigned to me? |
@yochannah I am an outreachy applicant and I am interested to pick up this issue. Can I work on this issue? |
@yochannah I used this package testipynb. I added a script that tests all *.ipynb files and reports whether they are failed or passed.
And here is how the output looks like: |
@niveditarufus @mkhoshle Both of you can try solving this issue. We won't judge you based on who has the best implementation here. The goal here is to demonstrate your collaboration skills(git, PR, communication). |
Submitted a pull request which might close the issue. @leoank @uosl @yochannah Kindly review it. |
Gitlab CI layer for fixing issue #43
APIs and data change, and over time our notebooks end up with errors. #42 is one example of this.
Let's add a CI layer that runs the notebooks periodically, so we can be notified when things break, rather than finding out at or soon before a tutorial when we're trying to teach them....
This may require some creative thinking to get the tokens working correctly.
Several great suggestions how to get the notebooks running in Travis or GitLab here: https://twitter.com/yoyehudi/status/1143118846234173440
The text was updated successfully, but these errors were encountered: