Skip to content
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 links to main repo examples from TF notebooks #398

Merged
merged 4 commits into from
Jun 20, 2023

Conversation

Rocketknight1
Copy link
Member

Some of the links in our TF examples went out of date! This PR fixes them. Fixes huggingface/transformers#24341

Thanks to @SoyGema for notifying us about the issue!

@Rocketknight1 Rocketknight1 requested a review from sgugger June 19, 2023 17:07
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@SoyGema
Copy link

SoyGema commented Jun 19, 2023

Hey :)
Thanks for the mention @Rocketknight1 and my apologies for not submitting the issue here!
Will definitely take it into account for the future

Keep up the good work :)

Copy link
Contributor

@sgugger sgugger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fixes!

@Rocketknight1 Rocketknight1 merged commit c6301af into main Jun 20, 2023
@Rocketknight1 Rocketknight1 deleted the fix_tf_notebook_links branch June 20, 2023 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Colab Translation notebook link not found
3 participants