Skip to content

Commit

Permalink
PR-template-int-test-update (#2112)
Browse files Browse the repository at this point in the history
* adding int test step to PR template
There is no associated ticket to this. So that is why the merge does not start with CMR-*
  • Loading branch information
jmaeng72 committed May 6, 2024
1 parent ab2e261 commit d3a9ce3
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,3 +21,7 @@ List impacted areas.
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [ ] My changes generate no new warnings
- [ ] I have cleaned up integration tests by doing one or more of the following:
- de-duped, consolidated, removed dead int tests
- transformed applicable int tests into unit tests
- refactored to reduce number of system state resets by updating fixtures (use-fixtures :each (ingest/reset-fixture {})) to be :once instead of :each

0 comments on commit d3a9ce3

Please sign in to comment.