mumrah commented on code in PR #18299:
URL: https://github.com/apache/kafka/pull/18299#discussion_r1983640140
##########
committer-tools/verify_license.py:
##########
@@ -77,14 +77,14 @@ def get_license_deps(license_text):
def main():
# Argument parser
parser = argparse.ArgumentParser(description="Whether to skip executing
ReleaseTarGz.")
- parser.add_argument('--skipReleaseTarGz', type=str, default=False,
help='Whether to skip ReleaseTarGz (true/false)')
+ parser.add_argument('--skip-build', action="store_true", help='Whether to
skip ReleaseTarGz (true/false)')
Review Comment:
nit: mixture of single and double quotes. I'd suggest using double quotes
like the rest of the file.
Also the `(true/false)` part is not necessary. The help text should say what
setting this flag does, e.g. "skip the build"
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
