Browse Source

[git_auth] Emphasize success

People assume the tool has failed and keep trying to re-run it when it
says to run git credential-luci login.

Bug: 408501013
Change-Id: Id1f671bacfdc19d4614ff03cfa69bfd484ad741b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/6441970
Reviewed-by: Yiwei Zhang <yiwzhang@google.com>
Commit-Queue: Allen Li <ayatane@chromium.org>
Allen Li 4 months ago
parent
commit
07d24776f2
1 changed files with 5 additions and 1 deletions
  1. 5 1
      git_auth.py

+ 5 - 1
git_auth.py

@@ -492,6 +492,8 @@ class ConfigWizard(object):
                 'Looks like we are running outside of a Gerrit repository,')
                 'Looks like we are running outside of a Gerrit repository,')
             self._println('so we will check your global Git configuration.')
             self._println('so we will check your global Git configuration.')
             self._run_outside_repo()
             self._run_outside_repo()
+        self._println()
+        self._println('Successfully finished auth configuration check.')
         self._print_actions_for_user()
         self._print_actions_for_user()
 
 
     def _run_outside_repo(self) -> None:
     def _run_outside_repo(self) -> None:
@@ -851,7 +853,9 @@ class ConfigWizard(object):
         if not self._user_actions:
         if not self._user_actions:
             return
             return
         self._println()
         self._println()
-        self._println('Things you need to do:')
+        self._println(
+            "However, there are some manual actions that are suggested")
+        self._println("(you don't have to re-run this command afterward):")
         for s in self._user_actions:
         for s in self._user_actions:
             self._println(f'- {s}')
             self._println(f'- {s}')