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

Enhance _get_all_bundles_info to load auth token from env varible #6880

Closed
yiheng-wang-nv opened this issue Aug 17, 2023 · 1 comment
Closed

Comments

@yiheng-wang-nv
Copy link
Contributor

It would be more convenient to receive not only an arg of auth_token, but also support to load from env variable.
More bundles will require more times of requests, and easier to reach to rate limit, therefore enabling the token from an env variable is helpful to reduce the rate limit issue in some places such as running the CI tests.

cc @wyli

@wyli
Copy link
Contributor

wyli commented Sep 22, 2023

this is not required since #6997, we can revisit if there are other cases using an auth_token.

@wyli wyli closed this as not planned Won't fix, can't repro, duplicate, stale Sep 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants