coryhouse / coryhouse/reactjsconsulting

Definition of done

Open
#145 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
374
Forks
33
PR merge metrics
No merged PRs in 30d

Description

I use a short, high-level checklist:

- [ ] DRY
- [ ] Usable
- [ ] Secure
- [ ] Tested
- [ ] Robust
- [ ] Readable
- [ ] Accessible
- [ ] Performant
- [ ] Configurable
- [ ] Documented
- [ ] Meets requirements

Notice that I have to run the code to complete this checklist. It can’t be automated.

If the check can be automated, I automate it.

I automatically inject this checklist into each pull request via [PULL_REQUEST_TEMPLATE.md](https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/creating-a-pull-request-template-for-your-repository)

The checklist needs to be short enough that I consistently use it, but long enough that it's thorough. This idea is inspired by [The Checklist Manifesto](https://www.amazon.com/Checklist-Manifesto-How-Things-Right/dp/0312430000).

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.