From 3a0a145b0ec52dff88b021c5937925f25294a10f Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Fri, 20 May 2022 12:52:33 -0400 Subject: upload: move label validation to core function This way we know we don't need to encode the labels. Change-Id: Ib83ed8f4ed05f00b9d2d06a9dd3f304e4443430e Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/337518 Tested-by: Mike Frysinger Reviewed-by: LaMont Jones --- subcmds/upload.py | 6 ------ 1 file changed, 6 deletions(-) (limited to 'subcmds/upload.py') diff --git a/subcmds/upload.py b/subcmds/upload.py index ef3d8e9d..20b8fe5d 100644 --- a/subcmds/upload.py +++ b/subcmds/upload.py @@ -421,12 +421,6 @@ Gerrit Code Review: https://www.gerritcodereview.com/ labels = set(_ExpandCommaList(branch.project.config.GetString(key))) for label in opt.labels: labels.update(_ExpandCommaList(label)) - # Basic sanity check on label syntax. - for label in labels: - if not re.match(r'^.+[+-][0-9]+$', label): - print('repo: error: invalid label syntax "%s": labels use forms ' - 'like CodeReview+1 or Verified-1' % (label,), file=sys.stderr) - sys.exit(1) # Handle e-mail notifications. if opt.notify is False: -- cgit v1.2.3-54-g00ecf