Skip to content

8999 - Checksum checker email reports mention a checksum checker opti…#1

Open
aroman-arvo wants to merge 2 commits intomasterfrom
8999
Open

8999 - Checksum checker email reports mention a checksum checker opti…#1
aroman-arvo wants to merge 2 commits intomasterfrom
8999

Conversation

@aroman-arvo
Copy link
Copy Markdown
Collaborator

References

  • Fixes #8999

Description

Error in a string

Instructions for Reviewers

Wrong launcher command was indicated

Checklist

  • My PR is small in size (e.g. less than 1,000 lines of code, not including comments & integration tests). Exceptions may be made if previously agreed upon.
  • My PR passes Checkstyle validation based on the Code Style Guide.
  • My PR includes Javadoc for all new (or modified) public methods and classes. It also includes Javadoc for large or complex private methods.
  • My PR passes all tests and includes new/updated Unit or Integration Tests based on the Code Testing Guide.
  • If my PR includes new libraries/dependencies (in any pom.xml), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
  • If my PR modifies REST API endpoints, I've opened a separate REST Contract PR related to this change.
  • If my PR includes new configurations, I've provided basic technical documentation in the PR itself.
  • If my PR fixes an issue ticket, I've linked them together.

…on -u, that does not exist in code or documentation
@aroman-arvo
Copy link
Copy Markdown
Collaborator Author

It seems that the option refers to the DailyReportEmailer (the checker-emailer command)
I send a pull request that fix the text

@MW3000
Copy link
Copy Markdown
Contributor

MW3000 commented Oct 2, 2023

While org.dspace.checker.DailyReportEmailer has a -u option, this -u option does not "add these bitstreams to be checked". So I think this does not fix this issue.

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.

Checksum checker email reports mention a checksum checker option -u, that does not exist in code or documentation

2 participants