Skip to content

Commit

Permalink
Switch to checkbox for deployment, remove tests check
Browse files Browse the repository at this point in the history
Switched the heading for deployment changes to a checkbox
under the checklist regarding what the effect of the changes
are.

Removed the check for tests since as of now we have test
automation that notifies the committer that their changes
break existing tests.
  • Loading branch information
lenikadali committed Oct 22, 2024
1 parent a9fd039 commit 453f671
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
- [ ] I have commented my code, particularly in hard-to-understand areas,
- [ ] I have made corresponding changes to the documentation,
- [ ] I have added tests that prove my fix is effective or that my feature works,
- [ ] New and existing unit tests pass locally with my changes
- [ ] Title include "WIP" if work is in progress.
-->

Expand All @@ -33,6 +32,7 @@ Include anything else we should know about.-->
- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)
- [ ] This change requires a change to the deployment process
- [ ] This change requires a documentation update
- [ ] Documentation update

Expand Down

0 comments on commit 453f671

Please sign in to comment.