Suggest using ggshield config set instance
at login
#684
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.
Context
To login on an instance other than the default instance, users pass the
--instance INSTANCE_URL
option toggshield auth login
. This does not make INSTANCE_URL the default instance though, leading to confusion.This PR makes
ggshield auth login
print a message suggesting to useggshield config set instance INSTANCE_URL
whenauth login
is called with--instance
.What has been done
Config.set_cmdline_instance()
into acmdline_instance
propertyDemo
Logging in on the default instance, no message:
Logging in on a custom instance, suggestion message is printed: