diff --git a/OWNERS b/OWNERS index 97bd9ef01..3a2ad6bec 100644 --- a/OWNERS +++ b/OWNERS @@ -1,6 +1,5 @@ set noparent -agable@chromium.org apolito@google.com dpranke@chromium.org ehmaldonado@chromium.org diff --git a/gclient.py b/gclient.py index b2f15a772..5acbfae1d 100755 --- a/gclient.py +++ b/gclient.py @@ -2698,11 +2698,6 @@ def CMDsync(parser, args): parser.add_option('--lock_timeout', type='int', default=5000, help='GIT ONLY - Deadline (in seconds) to wait for git ' 'cache lock to become available. Default is %default.') - # TODO(agable): Remove these when the oldest CrOS release milestone is M56. - parser.add_option('-t', '--transitive', action='store_true', - help='DEPRECATED: This is a no-op.') - parser.add_option('-m', '--manually_grab_svn_rev', action='store_true', - help='DEPRECATED: This is a no-op.') parser.add_option('--no-rebase-patch-ref', action='store_false', dest='rebase_patch_ref', default=True, help='Bypass rebase of the patch ref after checkout.') diff --git a/gclient_utils.py b/gclient_utils.py index 498b27a20..8267b9662 100644 --- a/gclient_utils.py +++ b/gclient_utils.py @@ -425,7 +425,6 @@ class Annotated(Wrapper): # Continue lockless. obj[0] += out while True: - # TODO(agable): find both of these with a single pass. cr_loc = obj[0].find(b'\r') lf_loc = obj[0].find(b'\n') if cr_loc == lf_loc == -1: diff --git a/git_cl.py b/git_cl.py index 4d999ed28..612d4f94d 100755 --- a/git_cl.py +++ b/git_cl.py @@ -2300,7 +2300,7 @@ class Changelist(object): if self.GetIssue() and (reviewers or cc): # GetIssue() is not set in case of non-squash uploads according to tests. - # TODO(agable): non-squash uploads in git cl should be removed. + # TODO(crbug.com/751901): non-squash uploads in git cl should be removed. gerrit_util.AddReviewers( self._GetGerritHost(), self._GerritChangeIdentifier(), diff --git a/presubmit_canned_checks.py b/presubmit_canned_checks.py index ef1984e8b..595971db7 100644 --- a/presubmit_canned_checks.py +++ b/presubmit_canned_checks.py @@ -1612,7 +1612,6 @@ def CheckLucicfgGenOutput(input_api, output_api, entry_script): output_api.PresubmitError) ] -# TODO(agable): Add this to PanProjectChecks. def CheckJsonParses(input_api, output_api): """Verifies that all JSON files at least parse as valid JSON.""" import json diff --git a/presubmit_support.py b/presubmit_support.py index ffcfe9f3f..f58530e0d 100755 --- a/presubmit_support.py +++ b/presubmit_support.py @@ -1119,12 +1119,12 @@ class Change(object): def TBRsFromDescription(self): """Returns all TBR reviewers listed in the commit description.""" tags = [r.strip() for r in self.tags.get('TBR', '').split(',') if r.strip()] - # TODO(agable): Remove support for 'Tbr:' when TBRs are programmatically - # determined by self-CR+1s. + # TODO(crbug.com/839208): Remove support for 'Tbr:' when TBRs are + # programmatically determined by self-CR+1s. footers = self.GitFootersFromDescription().get('Tbr', []) return sorted(set(tags + footers)) - # TODO(agable): Delete these once we're sure they're unused. + # TODO(crbug.com/753425): Delete these once we're sure they're unused. @property def BUG(self): return ','.join(self.BugsFromDescription())