Skip to content

Fix links to main repo examples from TF notebooks#398

Merged
Rocketknight1 merged 4 commits intomainfrom
fix_tf_notebook_links
Jun 20, 2023
Merged

Fix links to main repo examples from TF notebooks#398
Rocketknight1 merged 4 commits intomainfrom
fix_tf_notebook_links

Conversation

@Rocketknight1
Copy link
Copy Markdown
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
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@SoyGema
Copy link
Copy Markdown

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
Copy Markdown
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