From 76cc115b7a3c1e833e1155f9056502c03f10d8ce Mon Sep 17 00:00:00 2001 From: Milind Shakya Date: Tue, 17 Nov 2020 15:41:45 -0500 Subject: [PATCH] Add verbosity --- gitown/gitown.py | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/gitown/gitown.py b/gitown/gitown.py index 3b3ba61..b64b578 100644 --- a/gitown/gitown.py +++ b/gitown/gitown.py @@ -18,7 +18,8 @@ def __init__( files, owners, ownership_threshold=DEFAULT_OWNERSHIP_THRESHOLD, - codeowners_filename=DEFAULT_CODEOWNERS_FILE + codeowners_filename=DEFAULT_CODEOWNERS_FILE, + verbose=False ): self.files = files, self.original_codeowner_data = {} @@ -27,6 +28,7 @@ def __init__( self.owners = owners self.ownership_threshold = ownership_threshold self.codeowners_file = codeowners_filename + self.verbose = verbose with open(self.codeowners_file, newline='') as csvfile: reader = csv.reader(csvfile, delimiter=' ') @@ -62,11 +64,15 @@ def update_file(self, updated_data): csvfile.write("#\n") csvfile.write("# Order is important. The last matching pattern has the most precedence.\n") csvfile.write("\n") + csvfile.write("\n") csvfile.write("# This file is also being managed automatically by the gitown tool.\n") - + writer = csv.writer(csvfile, delimiter=' ', lineterminator='\n') for key, value in updated_data.items(): writer.writerow([key] + value) + if self.verbose: + print(f"updated data: {updated_data}") + print(f"original data: {self.original_codeowner_data}") self.updated = True def get_committer_line_frequency_percentage(self, committer_email, filename): @@ -107,10 +113,12 @@ def main(): parser.add_argument('filenames', nargs='+') parser.add_argument('--ownership_threshold') parser.add_argument('--codeowners_filename') + parser.add_argument('--verbose', '-v', action='count', default=0) args = parser.parse_args() files = args.filenames[0] ownership_threshold = int(args.ownership_threshold or DEFAULT_OWNERSHIP_THRESHOLD) codeowners_filename = args.codeowners_filename + verbose = bool(args.verbose) if len(files) == 0: parser.error('No filenames provided') @@ -125,7 +133,8 @@ def main(): files, owners, ownership_threshold=ownership_threshold, - codeowners_filename=codeowners_filename or DEFAULT_CODEOWNERS_FILE + codeowners_filename=codeowners_filename or DEFAULT_CODEOWNERS_FILE, + verbose=verbose ) codeowners.main()