Skip to content

Commit

Permalink
Make unit testing (usually) mandatory (run-llama#16371)
Browse files Browse the repository at this point in the history
  • Loading branch information
logan-markewich authored and jzhao62 committed Oct 4, 2024
1 parent 341b4c5 commit 828a21e
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
Expand Up @@ -29,11 +29,10 @@ Please delete options that are not relevant.

## How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Your pull-request will likely not be merged unless it is covered by some form of impactful unit testing.

- [ ] Added new unit/integration tests
- [ ] Added new notebook (that tests end-to-end)
- [ ] I stared at the code and made sure it makes sense
- [ ] I added new unit tests to cover this change
- [ ] I believe this change is already covered by existing unit tests

## Suggested Checklist:

Expand Down

0 comments on commit 828a21e

Please sign in to comment.