Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sort results in fixity report email #407

Merged
merged 2 commits into from
Oct 16, 2024

Conversation

Dhwaniartefact
Copy link
Contributor

@Dhwaniartefact Dhwaniartefact commented Sep 25, 2024

`--timestamps` will display fixity scan timestamp of each AIP.
`--sort` to display the result of fixity check error messages on top and success message on below.
@replaceafill replaceafill changed the title Remove ts from fixity cron template Sort results in fixity report email Sep 25, 2024
Copy link
Contributor

@replaceafill replaceafill left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Let's wait for @amayita's feedback before merging.

Copy link
Member

@amayita amayita left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Dhwaniartefact Dhwaniartefact merged commit 2996265 into qa/1.x Oct 16, 2024
@Dhwaniartefact Dhwaniartefact deleted the dev/issue-remove-ts-from-fixity-cron branch October 16, 2024 21:13
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.

3 participants