Reformat python files with black
Reformat python files with the black formatter [1].
[1] https://github.com/psf/black
Change-Id: I191bee9b9f0edfcc9c9309a7ef93ee96a852613b
diff --git a/abandon_stale.py b/abandon_stale.py
index 2e01131..bef1733 100755
--- a/abandon_stale.py
+++ b/abandon_stale.py
@@ -56,65 +56,118 @@
def _main():
parser = optparse.OptionParser()
- parser.add_option('-g', '--gerrit-url', dest='gerrit_url',
- metavar='URL',
- default=None,
- help='gerrit server URL')
- parser.add_option('-b', '--basic-auth', dest='basic_auth',
- action='store_true',
- help='(deprecated) use HTTP basic authentication instead'
- ' of digest')
- parser.add_option('-d', '--digest-auth', dest='digest_auth',
- action='store_true',
- help='use HTTP digest authentication instead of basic')
- parser.add_option('-n', '--dry-run', dest='dry_run',
- action='store_true',
- help='enable dry-run mode: show stale changes but do '
- 'not abandon them')
- parser.add_option('-t', '--test', dest='testmode', action='store_true',
- help='test mode: query changes with the `test-abandon` '
- 'topic and ignore age option')
- parser.add_option('-a', '--age', dest='age',
- metavar='AGE',
- default="6months",
- help='age of change since last update in days, months'
- ' or years (default: %default)')
- parser.add_option('-m', '--message', dest='message',
- metavar='STRING', default=None,
- help='custom message to append to abandon message')
- parser.add_option('--branch', dest='branches', metavar='BRANCH_NAME',
- default=[], action='append',
- help='abandon changes only on the given branch')
- parser.add_option('--exclude-branch', dest='exclude_branches',
- metavar='BRANCH_NAME',
- default=[],
- action='append',
- help='do not abandon changes on given branch')
- parser.add_option('--project', dest='projects', metavar='PROJECT_NAME',
- default=[], action='append',
- help='abandon changes only on the given project')
- parser.add_option('--exclude-project', dest='exclude_projects',
- metavar='PROJECT_NAME',
- default=[],
- action='append',
- help='do not abandon changes on given project')
- parser.add_option('--owner', dest='owner',
- metavar='USERNAME',
- default=None,
- action='store',
- help='only abandon changes owned by the given user')
- parser.add_option('--exclude-wip', dest='exclude_wip',
- action='store_true',
- help='Exclude changes that are Work-in-Progress')
- parser.add_option('-v', '--verbose', dest='verbose',
- action='store_true',
- help='enable verbose (debug) logging')
+ parser.add_option(
+ "-g",
+ "--gerrit-url",
+ dest="gerrit_url",
+ metavar="URL",
+ default=None,
+ help="gerrit server URL",
+ )
+ parser.add_option(
+ "-b",
+ "--basic-auth",
+ dest="basic_auth",
+ action="store_true",
+ help="(deprecated) use HTTP basic authentication instead of digest",
+ )
+ parser.add_option(
+ "-d",
+ "--digest-auth",
+ dest="digest_auth",
+ action="store_true",
+ help="use HTTP digest authentication instead of basic",
+ )
+ parser.add_option(
+ "-n",
+ "--dry-run",
+ dest="dry_run",
+ action="store_true",
+ help="enable dry-run mode: show stale changes but do not abandon them",
+ )
+ parser.add_option(
+ "-t",
+ "--test",
+ dest="testmode",
+ action="store_true",
+ help="test mode: query changes with the `test-abandon` "
+ "topic and ignore age option",
+ )
+ parser.add_option(
+ "-a",
+ "--age",
+ dest="age",
+ metavar="AGE",
+ default="6months",
+ help="age of change since last update in days, months"
+ " or years (default: %default)",
+ )
+ parser.add_option(
+ "-m",
+ "--message",
+ dest="message",
+ metavar="STRING",
+ default=None,
+ help="custom message to append to abandon message",
+ )
+ parser.add_option(
+ "--branch",
+ dest="branches",
+ metavar="BRANCH_NAME",
+ default=[],
+ action="append",
+ help="abandon changes only on the given branch",
+ )
+ parser.add_option(
+ "--exclude-branch",
+ dest="exclude_branches",
+ metavar="BRANCH_NAME",
+ default=[],
+ action="append",
+ help="do not abandon changes on given branch",
+ )
+ parser.add_option(
+ "--project",
+ dest="projects",
+ metavar="PROJECT_NAME",
+ default=[],
+ action="append",
+ help="abandon changes only on the given project",
+ )
+ parser.add_option(
+ "--exclude-project",
+ dest="exclude_projects",
+ metavar="PROJECT_NAME",
+ default=[],
+ action="append",
+ help="do not abandon changes on given project",
+ )
+ parser.add_option(
+ "--owner",
+ dest="owner",
+ metavar="USERNAME",
+ default=None,
+ action="store",
+ help="only abandon changes owned by the given user",
+ )
+ parser.add_option(
+ "--exclude-wip",
+ dest="exclude_wip",
+ action="store_true",
+ help="Exclude changes that are Work-in-Progress",
+ )
+ parser.add_option(
+ "-v",
+ "--verbose",
+ dest="verbose",
+ action="store_true",
+ help="enable verbose (debug) logging",
+ )
(options, _args) = parser.parse_args()
level = logging.DEBUG if options.verbose else logging.INFO
- logging.basicConfig(format='%(asctime)s %(levelname)s %(message)s',
- level=level)
+ logging.basicConfig(format="%(asctime)s %(levelname)s %(message)s", level=level)
if not options.gerrit_url:
logging.error("Gerrit URL is required")
@@ -128,8 +181,10 @@
if not match:
logging.error("Invalid age: %s", options.age)
return 1
- message = "Abandoning after %s %s or more of inactivity." % \
- (match.group(1), match.group(2))
+ message = "Abandoning after %s %s or more of inactivity." % (
+ match.group(1),
+ match.group(2),
+ )
if options.digest_auth:
auth_type = HTTPDigestAuthFromNetrc
@@ -210,8 +265,10 @@
continue
try:
- gerrit.post("/changes/" + change_id + "/abandon",
- json={"message": "%s" % abandon_message})
+ gerrit.post(
+ "/changes/" + change_id + "/abandon",
+ json={"message": "%s" % abandon_message},
+ )
abandoned += 1
except Exception as e:
errors += 1
diff --git a/release-announcement.py b/release-announcement.py
index 7b74770..688de42 100755
--- a/release-announcement.py
+++ b/release-announcement.py
@@ -70,7 +70,7 @@
class Version:
def __init__(self, version):
self.version = version
- parts = version.split('.')
+ parts = version.split(".")
if len(parts) > 2:
self.major = ".".join(parts[:2])
self.patch = version.replace(".", "")
@@ -83,17 +83,26 @@
def _main():
- descr = 'Generate Gerrit release announcement email text'
+ descr = "Generate Gerrit release announcement email text"
parser = argparse.ArgumentParser(
- description=descr,
- formatter_class=argparse.ArgumentDefaultsHelpFormatter)
- parser.add_argument('-v', '--version', dest='version',
- required=True,
- help='gerrit version to release')
- parser.add_argument('-p', '--previous', dest='previous',
- help='previous gerrit version (optional)')
- parser.add_argument('-s', '--summary', dest='summary',
- help='summary of the release content (optional)')
+ description=descr, formatter_class=argparse.ArgumentDefaultsHelpFormatter
+ )
+ parser.add_argument(
+ "-v",
+ "--version",
+ dest="version",
+ required=True,
+ help="gerrit version to release",
+ )
+ parser.add_argument(
+ "-p", "--previous", dest="previous", help="previous gerrit version (optional)"
+ )
+ parser.add_argument(
+ "-s",
+ "--summary",
+ dest="summary",
+ help="summary of the release content (optional)",
+ )
options = parser.parse_args()
summary = options.summary
@@ -103,22 +112,26 @@
data = {
"version": Version(options.version),
"previous": options.previous,
- "summary": summary
+ "summary": summary,
}
war = os.path.join(
os.path.expanduser("~/.m2/repository/com/google/gerrit/gerrit-war/"),
- "%(version)s/gerrit-war-%(version)s.war" % data)
+ "%(version)s/gerrit-war-%(version)s.war" % data,
+ )
if not os.path.isfile(war):
- print("Could not find war file for Gerrit %s in local Maven repository"
- % data["version"], file=sys.stderr)
+ print(
+ "Could not find war file for Gerrit %s in local Maven repository"
+ % data["version"],
+ file=sys.stderr,
+ )
sys.exit(1)
md5 = hashlib.md5()
sha1 = hashlib.sha1()
sha256 = hashlib.sha256()
BUF_SIZE = 65536 # Read data in 64kb chunks
- with open(war, 'rb') as f:
+ with open(war, "rb") as f:
while True:
d = f.read(BUF_SIZE)
if not d:
@@ -132,8 +145,8 @@
data["md5"] = md5.hexdigest()
template_path = os.path.join(
- os.path.dirname(os.path.realpath(__file__)),
- "release-announcement-template.txt")
+ os.path.dirname(os.path.realpath(__file__)), "release-announcement-template.txt"
+ )
template = Template(open(template_path).read())
output = template.render(data=data)
