-
Notifications
You must be signed in to change notification settings - Fork 2k
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
chore: remove tutorials from the repo #3244
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
73c0867
to
b23c1dc
Compare
Looks good, however the legacy tutorials under docs/ versions have colab buttons that point to these ones. They either have to be updated here or in haystack-website (although the repo is ignored in haystack-website). Let me know which way to go about it and I'll be happy to update them |
@TuanaCelik do you have an easy way to search and replace all the old versions? If yes you can open a PR with the fix, we merge and I rebuild the old website one last time before switching to the new build script |
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.
Looking good to me! I found one minuscule possible improvement but nothing important.
<a href="https://github.com/deepset-ai/haystack/actions/workflows/tutorials_nightly.yml"> | ||
<img alt="Tutorials" src="https://github.com/deepset-ai/haystack/actions/workflows/tutorials_nightly.yml/badge.svg"> | ||
</a> |
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.
Do you think we can replace this with a badge from the other repo?
<a href="https://github.com/deepset-ai/haystack/actions/workflows/tutorials_nightly.yml"> | |
<img alt="Tutorials" src="https://github.com/deepset-ai/haystack/actions/workflows/tutorials_nightly.yml/badge.svg"> | |
</a> | |
<a href="https://github.com/deepset-ai/haystack/actions/workflows/tutorials_nightly.yml"> | |
<img alt="Tutorials" src="https://github.com/deepset-ai/haystack-tutorials/actions/workflows/nightly.yml/badge.svg"> | |
</a> |
Proposed Changes:
Moving tutorials to https://github.com/deepset-ai/haystack-tutorials
How did you test it?
Notes for the reviewer
Checklist