mirror of
https://chromium.googlesource.com/chromium/tools/depot_tools.git
synced 2026-01-11 18:51:29 +00:00
git_cl: Delete usages of AutoConfigure
Has been no-op for a while, remove it now Bug: 446999231 Change-Id: I9f4e5b9cbd6a5cb8e84fba2f0ebb949b2a88d9ff Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/6981220 Reviewed-by: Gavin Mak <gavinmak@google.com> Commit-Queue: Allen Li <ayatane@chromium.org>
This commit is contained in:
13
git_auth.py
13
git_auth.py
@@ -261,19 +261,6 @@ class ConfigChanger(object):
|
||||
self._set_config_func(*args, **kwargs)
|
||||
|
||||
|
||||
def AutoConfigure(cwd: str, cl: git_cl.Changelist) -> None:
|
||||
"""Configure Git authentication automatically.
|
||||
|
||||
This tracks when the config that has already been applied and skips
|
||||
doing anything if so.
|
||||
|
||||
This may modify the global Git config and the local repo config as
|
||||
needed.
|
||||
"""
|
||||
# TODO(ayatane): Disable prior to removal
|
||||
return
|
||||
|
||||
|
||||
def Configure(cwd: str, cl: git_cl.Changelist) -> None:
|
||||
"""Configure Git authentication.
|
||||
|
||||
|
||||
@@ -2397,7 +2397,6 @@ class Changelist(object):
|
||||
return
|
||||
|
||||
if newauth.Enabled():
|
||||
git_auth.AutoConfigure(os.getcwd(), Changelist())
|
||||
return
|
||||
|
||||
# Lazy-loader to identify Gerrit and Git hosts.
|
||||
|
||||
@@ -2445,7 +2445,6 @@ class TestGitCl(unittest.TestCase):
|
||||
mock.patch(
|
||||
'git_cl.gerrit_util.CookiesAuthenticator',
|
||||
CookiesAuthenticatorMockFactory(hosts_with_creds=auth)).start()
|
||||
mock.patch('git_auth.AutoConfigure', return_value=None).start()
|
||||
scm.GIT.SetConfig('', 'remote.origin.url',
|
||||
'https://chromium.googlesource.com/my/repo')
|
||||
cl = git_cl.Changelist()
|
||||
|
||||
Reference in New Issue
Block a user