cisagov / tpt-reports

Process to build and distribute Technical Phishing Test (TPT) reports
Creative Commons Zero v1.0 Universal
4 stars 0 forks source link

Update the output log message with more descriptive qualities #46

Closed ameliav closed 12 months ago

ameliav commented 12 months ago

๐Ÿ—ฃ Description

This PR resolves issue https://github.com/cisagov/tpt-reports/issues/43 and adds a more descriptive output log message in CLI for the generated report.

๐Ÿ’ญ Motivation and context

To make the log message unique to the assessment and location of the output directory.

๐Ÿงช Testing

Run the script as usage suggests and verify output references are replaced.

โœ… Pre-approval checklist

โœ… Pre-merge checklist

โœ… Post-merge checklist

coveralls commented 12 months ago

Pull Request Test Coverage Report for Build 6226865899


Totals Coverage Status
Change from base Build 6226574428: 0.0%
Covered Lines: 0
Relevant Lines: 0

๐Ÿ’› - Coveralls
dav3r commented 12 months ago

Reminder that if a checklist item in the PR template does not apply, you should delete it, rather than check the box (which implies that something was done for that item). In this case, you can delete the "Revert dependencies to default branches." item because this PR did not require the use of any test branches of dependent repositories.

Also, I unchecked the "Finalize version." checkbox because that has not been done yet for this PR. If you're unclear about what this means, please see https://github.com/cisagov/tpt-reports/pull/37#issuecomment-1720055509. If you have any further questions, I'm happy to answer them.