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

chore: rename api_key to hf_token #6437

Merged
merged 4 commits into from Nov 15, 2023
Merged

Conversation

NickCrews
Copy link
Contributor

@NickCrews NickCrews commented Nov 15, 2023

The param name changed in the past but these comments and error messages are stale.

Closes: NA

The param name changed in the past but these comments and error
messages are stale.
@abidlabs abidlabs added the no-visual-update Add this to a PR to skip chromatic deployment and tests label Nov 15, 2023
@gradio-pr-bot
Copy link
Contributor

gradio-pr-bot commented Nov 15, 2023

🪼 branch checks and previews

Name Status URL
🦄 Changes detected! Details
📓 Notebooks not matching! Details

The demo notebooks don't match the run.py files. Please run this command from the root of the repo and then commit the changes:

pip install nbformat && cd demo && python generate_notebooks.py

@gradio-pr-bot
Copy link
Contributor

gradio-pr-bot commented Nov 15, 2023

🦄 change detected

This Pull Request includes changes to the following packages.

Package Version
gradio minor
  • Maintainers can select this checkbox to manually select packages to update.

With the following changelog entry.

chore: rename api_key to hf_token

Maintainers or the PR author can modify the PR title to modify this entry.

Something isn't right?

  • Maintainers can change the version label to modify the version bump.
  • If the bot has failed to detect any changes, or if this pull request needs to update multiple packages to different versions or requires a more comprehensive changelog entry, maintainers can update the changelog file directly.

@abidlabs
Copy link
Member

LGTM, thanks @NickCrews for the contribution! I also updated a few of our demo notebooks and added here.

@abidlabs abidlabs merged commit 727ae25 into gradio-app:main Nov 15, 2023
12 of 17 checks passed
@pngwn pngwn mentioned this pull request Nov 15, 2023
@NickCrews
Copy link
Contributor Author

Good catch, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-visual-update Add this to a PR to skip chromatic deployment and tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants