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

Added details around benchcab outputs to the PR template. Fixes #517 #518

Merged
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,11 @@ Please delete options that are not relevant.

## Checklist

- [ ] The new content is accessible and located in the appropriate section.
- [ ] I have checked that links are valid and point to the intended content.
- [ ] The new content is accessible and located in the appropriate section
- [ ] I have checked that links are valid and point to the intended content
- [ ] I have checked my code/text and corrected any misspellings
- [ ] I have included relevant test results (i.e. benchcab outputs or path to readable logs) to ensure that the code compiles, runs and is bitwise compatible to the main branch*

**Unless a a change in outputs is expected, in which case add someone with relevant scientific expertise as a reviewer*

Please add a reviewer when ready for review.
Loading