-
Notifications
You must be signed in to change notification settings - Fork 13
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
Update reviewing.md to reflect CNTI #90
Conversation
Refs lfn-cnti#77 Signed-off-by: Martin Matyas <martin.matyas@gmail.com>
90f2b43
to
644f147
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
1. Verify that the list of files matches the [expected list](https://github.com/lfn-cnti/certification/blob/main/instructions.md#contents-of-the-pr). | ||
1. Note the vX.Y subdirectory that the PR is in, this is the version of CNTI Certification for which certified is being claimed, referenced as the "Certification Version" from hereon. | ||
1. Verify that the Certification Version is the current or previous version of CNTI Certification. | ||
1. Look at results log, cnf-testsuite-results-YYYY-MMDD-HHMMSS-NNN.yml. Verify that the major.minor component of the CNTI Certification matches the certification version being claimed. The patch version does not matter. | ||
1. Verify that the RESULTS SUMMARY section of results log "X of Y essential tests passed". There should be at least 10 essential tests passed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI - this sentence "There should be at least 10 essential tests passed" will be updated in the next 1-2 weeks when the v2.0 certification list of tests is updated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changes requested
Co-authored-by: Lucina Stricko <lucina@vulk.coop>
Co-authored-by: Lucina Stricko <lucina@vulk.coop>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
* Update reviewing.md Refs #90 * Update reviewing.md
Refs #77
Pre-submission checklist:
Please check each of these after submitting your pull request: