Skip to content

Commit

Permalink
Log "PROGRAM COMPLETE" instead of "PROGRAM COMPLETED"
Browse files Browse the repository at this point in the history
- logging "PROGRAM COMPLETE" instead of "PROGRAM COMPLETED" aligns the text in the log file
  - since
    - STARTING PROGRAM
    - PROGRAM COMPLETE
    have the same number of characters
  - and they are the first and last thing the program logs, respectively
  - removing the "D" allows the first and last log messages to have exactly the same number of characters, so if a user runs the program on the same channel multiple times, the first and last log message for each run will be aligned
  • Loading branch information
shailshouryya committed Dec 31, 2020
1 parent e488352 commit ab18840
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion python/dev/execute.py
Expand Up @@ -177,5 +177,5 @@ def yield_logger(file_name):
program_end = time.perf_counter()
total_time = program_end - program_start
log(f'This program took {total_time} seconds to complete.', logging_locations)
log( '>' * 50 + 'PROGRAM COMPLETED' + '<' * 50, logging_locations)
log( '>' * 50 + 'PROGRAM COMPLETE' + '<' * 50, logging_locations)
return file_name
2 changes: 1 addition & 1 deletion python/yt_videos_list/execute.py
Expand Up @@ -141,5 +141,5 @@ def yield_logger(file_name):
program_end = time.perf_counter()
total_time = program_end - program_start
log(f'This program took {total_time} seconds to complete.', logging_locations)
log( '>' * 50 + 'PROGRAM COMPLETED' + '<' * 50, logging_locations)
log( '>' * 50 + 'PROGRAM COMPLETE' + '<' * 50, logging_locations)
return file_name

0 comments on commit ab18840

Please sign in to comment.