-
Notifications
You must be signed in to change notification settings - Fork 0
/
gitmessage
29 lines (29 loc) · 1007 Bytes
/
gitmessage
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
- [ ] Why
- [ ] Why am I doing this?
- [ ] Who is the target audience?
- [ ] How does it help them?
- [ ] Overall Approach
- [ ] Does this make it easier to change the product/software in the future?
- [ ] Principle of least surprise
- [ ] Is there anything not clear?
- [ ] Is there anything surprising?
- [ ] Is there anything that looks "wrong"?
- [ ] License
- [ ] Compatibility impact
- [ ] Are there any breaking changes?
- [ ] What can go wrong?
- [ ] Any accidental culture/language specific code
- C#: ~StartsWith~, ~EndsWith~
- [ ] Update packageIndex.json and packageVersion for runtime changes
- [ ] Readability (with comments for tricky aspects)
- [ ] Parallelism/Thread Safety
- [ ] Algorithms and boundry cases
- [ ] Performance
- [ ] Is this code performance sensitive?
- [ ] Do I have any benchmarks?
- [ ] Error Handling
- [ ] Resource Handling
- [ ] Unit/Integration Tests
- [ ] Documentation
- [ ] Does this need announcement?
- [ ] Release notes?