From fb8477d70c9f96bc96efcdfdce6d512961fc9efd Mon Sep 17 00:00:00 2001 From: Scott Gigante Date: Mon, 18 Dec 2023 22:37:27 -0500 Subject: [PATCH] Improve usability of license.py MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Removes unused arguments and marks required subparser as required. Before: ``` ✗ python scripts/license.py Traceback (most recent call last): File "/Users/scottgigante/git/ctvc/codecov/scripts/license.py", line 93, in main() File "/Users/scottgigante/git/ctvc/codecov/scripts/license.py", line 83, in main print(pref.command) ^^^^^^^^^^^^ AttributeError: 'Namespace' object has no attribute 'command' ``` After: ``` ✗ python scripts/license.py usage: codecov key gen [-h] {new,check} ... codecov key gen: error: the following arguments are required: {new,check} ``` --- scripts/license.py | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/scripts/license.py b/scripts/license.py index 86b5ee8..dd13d84 100644 --- a/scripts/license.py +++ b/scripts/license.py @@ -58,17 +58,10 @@ def decode_license(license): def main(args=None): - expires = None - company = None - seats = None - repos = None - license = None - url = None - trial = True parser = argparse.ArgumentParser(prog='codecov key gen', add_help=True, formatter_class=argparse.RawDescriptionHelpFormatter) - subparsers = parser.add_subparsers(title='Commands') + subparsers = parser.add_subparsers(title='Commands', required=True) make = subparsers.add_parser('new') make.add_argument('--expires', action="store", required=True, @@ -97,4 +90,4 @@ def main(args=None): if __name__ == "__main__": - main() \ No newline at end of file + main()