Skip to content

Added Summaries for the Count of Checks Accross All Matched Files 2#96

Draft
hankgref wants to merge 2 commits intomasterfrom
8-provide-summaries-for-the-counts-of-checks-across-all-matched-files
Draft

Added Summaries for the Count of Checks Accross All Matched Files 2#96
hankgref wants to merge 2 commits intomasterfrom
8-provide-summaries-for-the-counts-of-checks-across-all-matched-files

Conversation

@hankgref
Copy link
Copy Markdown
Collaborator

In order to provide a summary to the count of checks throughout the output, I added some lines of code to the end of the output which would display the subtotal of all of the checks. The code is as follows:

opt_print_log("Sub-total for the number of matches:")
opt_print_log((current_check._matches))

@hankgref hankgref linked an issue Oct 27, 2023 that may be closed by this pull request
@KellerLiptrap
Copy link
Copy Markdown
Collaborator

@hankgref I think this is a good start. You may want to add another print that will display the number of files in the output. Maybe using a loop to count each file.

@boulais01
Copy link
Copy Markdown
Collaborator

Hey @hankgref, please consider updating this to pass the checks and requesting review if this is complete.

@boulais01 boulais01 marked this pull request as draft November 17, 2023 14:35
@boulais01
Copy link
Copy Markdown
Collaborator

@hankgref Can you give on update on this PR?

1 similar comment
@boulais01
Copy link
Copy Markdown
Collaborator

@hankgref Can you give on update on this PR?

@Finley8
Copy link
Copy Markdown
Collaborator

Finley8 commented Dec 14, 2023

Hello Hank, I would love to see an update on this PR. Is there anything you would like some help with?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Provide Summaries for the Counts of Checks Across All Matched Files

4 participants