-
Notifications
You must be signed in to change notification settings - Fork 30
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
Fix broken links #564
Fix broken links #564
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #564 +/- ##
=======================================
Coverage 92.51% 92.51%
=======================================
Files 59 59
Lines 4393 4393
=======================================
Hits 4064 4064
Misses 329 329 ☔ View full report in Codecov by Sentry. |
Nice @nwlandry thank you! I have noticed that there are also links in the tutorials that were broken with the moving (I think all of them are in the Getting started tutorials. Lmk know if you want to do the changes yourself or I can also do it. |
@thomasrobiglio I won't have a chance to do this today or tomorrow, so feel free to make the changes, approve, and merge! If not, I'll get to it at some point. |
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Actually, I had a bit of time open up, so working on this now! |
All fixed, @thomasrobiglio! |
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.
Nice! All good, thank you also for adding the .ipynb
files to the urls-checking routing.
Thanks for the review! |
Fix the broken links since the tutorials were moved.