Skip to content

Latest commit

 

History

History
43 lines (22 loc) · 1.23 KB

pull_request_template.md

File metadata and controls

43 lines (22 loc) · 1.23 KB

DESCRIPTION

[Describe what is changed, including references to any relevant issues.]

New Tests

[List all new test cases added to cover new functionality, including steps to reproduce expected results and contrasting with actual results.]

Checklist

Function

[] Code is fully functional and ready for review (or is tagged WIP)

[] No errors/warnings in the browser console ( ゚ヮ゚)/

[] package.json is committed if I added or removed any dependencies

[] Tests added as needed to ensure code base still has 100% passing coverage

[] Tests added as needed to cover new functionality

Style

[] Passes linter without errors

[] Code is easily understandable and self-documenting (using naming conventions in keeping with the rest of the codebase)

[] Interfaces all have 'I' before their names (as in IAudit)

[] Comments are only included as needed to explain the 'why' instead of the 'what'

[] Common standards and coding practices are adhered to

[] DRY: Don't Repeat Yourself. Code is reasonably structured to avoid repetition

[] Code is intuitively structured following a Separation of Concerns

[] No class components; only functional components (using Hooks as needed)

Workflow

[] Do an interactive rebase or a rebase