[gerrit_util] Update docstring for new param [chromium/tools/depot_tools : main]

0 views
Skip to first unread message

Allen Li (Gerrit)

unread,
7:20 PM (4 hours ago) 7:20 PM
to chromium...@chromium.org, chops-source-team...@google.com

Allen Li has uploaded the change for review

Commit message

[gerrit_util] Update docstring for new param
Change-Id: Ifb6106ad0063183ee0a0377bb3564c4fc99d88ef

Change diff

diff --git a/gerrit_util.py b/gerrit_util.py
index a7619c3..323e269 100644
--- a/gerrit_util.py
+++ b/gerrit_util.py
@@ -223,7 +223,11 @@
@classmethod
def is_applicable(cls, *, conn: Optional[HttpConn] = None) -> bool:
"""Must return True if this Authenticator is available in the current
- environment."""
+ environment.
+
+ If conn is not None, return True if this Authenticator is
+ applicable to the given conn in the current environment.
+ """
raise NotImplementedError()

def ensure_authenticated(self, gerrit_host: str, git_host: str) -> Tuple[bool, str]:

Change information

Files:
  • M gerrit_util.py
Change size: XS
Delta: 1 file changed, 5 insertions(+), 1 deletion(-)
Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
  • requirement is not satisfiedCode-Review
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: newchange
Gerrit-Project: chromium/tools/depot_tools
Gerrit-Branch: main
Gerrit-Change-Id: Ifb6106ad0063183ee0a0377bb3564c4fc99d88ef
Gerrit-Change-Number: 5673779
Gerrit-PatchSet: 1
Gerrit-Owner: Allen Li <aya...@chromium.org>
unsatisfied_requirement
open
diffy

Allen Li (Gerrit)

unread,
7:21 PM (4 hours ago) 7:21 PM
to Robbie Iannucci, Yiwei Zhang, chromium...@chromium.org, chops-source-team...@google.com
Attention needed from Robbie Iannucci and Yiwei Zhang

Allen Li voted

Auto-Submit+1
Commit-Queue+1
Open in Gerrit

Related details

Attention is currently required from:
  • Robbie Iannucci
  • Yiwei Zhang
Submit Requirements:
  • requirement is not satisfiedCode-Review
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/tools/depot_tools
Gerrit-Branch: main
Gerrit-Change-Id: Ifb6106ad0063183ee0a0377bb3564c4fc99d88ef
Gerrit-Change-Number: 5673779
Gerrit-PatchSet: 1
Gerrit-Owner: Allen Li <aya...@chromium.org>
Gerrit-Reviewer: Allen Li <aya...@chromium.org>
Gerrit-Reviewer: Robbie Iannucci <iann...@chromium.org>
Gerrit-Reviewer: Yiwei Zhang <yiwz...@google.com>
Gerrit-Attention: Robbie Iannucci <iann...@chromium.org>
Gerrit-Attention: Yiwei Zhang <yiwz...@google.com>
Gerrit-Comment-Date: Tue, 02 Jul 2024 23:21:03 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
unsatisfied_requirement
open
diffy

Yiwei Zhang (Gerrit)

unread,
7:21 PM (4 hours ago) 7:21 PM
to Allen Li, LUCI CQ, Robbie Iannucci, chromium...@chromium.org, chops-source-team...@google.com
Attention needed from Allen Li and Robbie Iannucci

Yiwei Zhang voted Code-Review+1

Code-Review+1
Open in Gerrit

Related details

Attention is currently required from:
  • Allen Li
  • Robbie Iannucci
Submit Requirements:
  • requirement satisfiedCode-Review
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/tools/depot_tools
Gerrit-Branch: main
Gerrit-Change-Id: Ifb6106ad0063183ee0a0377bb3564c4fc99d88ef
Gerrit-Change-Number: 5673779
Gerrit-PatchSet: 1
Gerrit-Owner: Allen Li <aya...@chromium.org>
Gerrit-Reviewer: Allen Li <aya...@chromium.org>
Gerrit-Reviewer: Robbie Iannucci <iann...@chromium.org>
Gerrit-Reviewer: Yiwei Zhang <yiwz...@google.com>
Gerrit-Attention: Robbie Iannucci <iann...@chromium.org>
Gerrit-Attention: Allen Li <aya...@chromium.org>
Gerrit-Comment-Date: Tue, 02 Jul 2024 23:21:28 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Allen Li (Gerrit)

unread,
9:29 PM (2 hours ago) 9:29 PM
to Yiwei Zhang, LUCI CQ, Robbie Iannucci, chromium...@chromium.org, chops-source-team...@google.com
Attention needed from Robbie Iannucci

Allen Li voted Commit-Queue+2

Commit-Queue+2
Open in Gerrit

Related details

Attention is currently required from:
  • Robbie Iannucci
Submit Requirements:
  • requirement satisfiedCode-Review
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/tools/depot_tools
Gerrit-Branch: main
Gerrit-Change-Id: Ifb6106ad0063183ee0a0377bb3564c4fc99d88ef
Gerrit-Change-Number: 5673779
Gerrit-PatchSet: 1
Gerrit-Owner: Allen Li <aya...@chromium.org>
Gerrit-Reviewer: Allen Li <aya...@chromium.org>
Gerrit-Reviewer: Robbie Iannucci <iann...@chromium.org>
Gerrit-Reviewer: Yiwei Zhang <yiwz...@google.com>
Gerrit-Attention: Robbie Iannucci <iann...@chromium.org>
Gerrit-Comment-Date: Wed, 03 Jul 2024 01:29:12 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy
Reply all
Reply to author
Forward
0 new messages