Align HfFolder.get_token
with huggingface_hub.get_token
#1966
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Since
0.20.x
release,HfFolder
is planned to be deprecated in favor ofget_token
,login
andlogout
. Deprecation warning is not thrown yet asHfFolder
is still a widely used feature.@Narsil made me realize that the implementation of
HfFolder.get_token
andget_token
are slightly different. In particular, ifHF_TOKEN
environment variable is not set and a token is saved locally, thenHfFolder.get_token
will returnNone
(which is a bug) whileget_token
returns the saved token. In order to avoid any discrepancies, this PR replacesHfFolder.get_token
implementation by a call to the more robustget_token
.Note: this change brokes
test_token_in_old_path
mocked test. I decided to remove it completely as it's quite old stuff (13 months old) and by now we can assume all users have switched from the old path to the new one. The feature is still present, I just removed the test.