SHOW:
|
|
- or go back to the newest paste.
| 1 | - | ``` |
| 1 | + | ``` |
| 2 | - | As a developer |
| 2 | + | As a developer |
| 3 | - | I want to get better at making my PRs easier to review |
| 3 | + | I want to get better at making my PRs easier to review |
| 4 | - | So that they get reviewed sooner and I can merge faster => shorter cycle times |
| 4 | + | So that they get reviewed sooner and I can merge faster => shorter cycle times |
| 5 | - | |
| 5 | + | |
| 6 | - | As a code reviewer |
| 6 | + | As a code reviewer |
| 7 | - | I want PRs that are easier to review |
| 7 | + | I want PRs that are easier to review |
| 8 | - | So that it's less work |
| 8 | + | So that it's less work |
| 9 | - | ``` |
| 9 | + | ``` |
| 10 | - | |
| 10 | + | |
| 11 | - | I wish to claim that spending extra time as developers on writing PRs is repaid in time saved by the people reading them - if only because they are read more often than written. Here are some of the things I now try and remember to put in on each one. Obviously not all points are relevant to all changes in all projects. |
| 11 | + | I wish to claim that spending extra time as developers on writing PRs is repaid in time saved by the people reading them - if only because they are read more often than written. Here are some of the things I now try and remember to put in on each one. Obviously not all points are relevant to all changes in all projects. |
| 12 | - | |
| 12 | + | |
| 13 | I would like to hear your opinions: as a reviewer, are there other things you would really like to see in PR messages that you don't, or are there things I'm proposing to write here that you really don't think are valuable to read? As a contributer, does this seem too little/about right/too onerous? |
