@@ -36,7 +36,7 @@ def set_verbosity(enable_verbosity):
3636 """Turn debug output on or off.
3737
3838 Keyword arguments:
39- enable_verbosity -- this will generally be controlled via the script's -- verbose command line argument
39+ enable_verbosity -- enable/disable verbose output
4040 (True, False)
4141 """
4242 # DEBUG: automatically generated output and all higher log level output
@@ -123,9 +123,9 @@ def report_size_deltas_from_workflow_artifacts(self):
123123 pr_number = pr_data ["number" ]
124124 pr_head_sha = pr_data ["head" ]["sha" ]
125125 print ("::debug::Processing pull request number:" , pr_number )
126- # When a PR is locked, only collaborators may comment. The automatically generated GITHUB_TOKEN will
127- # likely be used, which is owned by the github-actions bot, who doesn't have collaborator status. So
128- # locking the thread would cause the job to fail .
126+ # When a PR is locked, only collaborators may comment. The automatically generated GITHUB_TOKEN owned by
127+ # the github-actions bot will likely be used. The bot doesn't have collaborator status so it will
128+ # generally be impossible to make reports on locked PRs .
129129 if pr_data ["locked" ]:
130130 print ("::debug::PR locked, skipping" )
131131 continue
@@ -280,7 +280,7 @@ def get_sketches_reports(self, artifact_folder_object):
280280 """Parse the artifact files and return a list containing the data.
281281
282282 Keyword arguments:
283- artifact_folder_object -- object containing the data about the temporary folder that stores the markdown files
283+ artifact_folder_object -- object containing the data about the temporary folder that stores the Markdown files
284284 """
285285 with artifact_folder_object as artifact_folder :
286286 # artifact_folder will be a string when running in non-local report mode
@@ -317,7 +317,7 @@ def generate_report(self, sketches_reports):
317317 """Return the Markdown for the deltas report comment.
318318
319319 Keyword arguments:
320- sketches_reports -- list of sketches_reports containing the data to generate the deltas report from
320+ sketches_reports -- list of sketches reports containing the data to generate the deltas report from
321321 """
322322 # From https://github.community/t/maximum-length-for-the-comment-body-in-issues-and-pr/148867/2
323323 # > PR body/Issue comments are still stored in MySQL as a mediumblob with a maximum value length of 262,144.
@@ -466,7 +466,7 @@ def get_summary_value(self, show_emoji, minimum, maximum):
466466 Keyword arguments:
467467 show_emoji -- whether to add the emoji change indicator
468468 minimum -- minimum amount of change for this memory type
469- minimum -- maximum amount of change for this memory type
469+ maximum -- maximum amount of change for this memory type
470470 """
471471 size_decrease_emoji = ":green_heart:"
472472 size_ambiguous_emoji = ":grey_question:"
@@ -550,7 +550,7 @@ def get_json_response(self, url):
550550 except json .decoder .JSONDecodeError as exception :
551551 # Output some information on the exception
552552 logger .warning (str (exception .__class__ .__name__ ) + ": " + str (exception ))
553- # pass on the exception to the caller
553+ # Pass the exception on to the caller
554554 raise exception
555555
556556 if not json_data :
@@ -722,13 +722,13 @@ def get_report_column_number(report, column_heading):
722722 # Absolute column
723723 # Add the heading
724724 report [0 ].append (column_heading )
725- # Expand the size of the last ( current) row to match the new number of columns
725+ # Expand the size of the final row (the current row) to match the new number of columns
726726 report [len (report ) - 1 ].append ("" )
727727
728728 # Relative column
729729 # Add the heading
730730 report [0 ].append (relative_column_heading )
731- # Expand the size of the last ( current) row to match the new number of columns
731+ # Expand the size of the final row (the current row) to match the new number of columns
732732 report [len (report ) - 1 ].append ("" )
733733
734734 return column_number
0 commit comments