reviewing-a-pull-request
Differences
This shows you the differences between two versions of the page.
| reviewing-a-pull-request [2023/09/20 15:40] – created Roman Sheydvasser | reviewing-a-pull-request [2023/12/19 14:39] (current) – Roman Sheydvasser | ||
|---|---|---|---|
| Line 18: | Line 18: | ||
| - If there aren’t automated tests, confirm that the PR description contains at least 1-2 items describing how the changes were tested. | - If there aren’t automated tests, confirm that the PR description contains at least 1-2 items describing how the changes were tested. | ||
| - Check each changed file for potential bugs that the submitter may have missed. | - Check each changed file for potential bugs that the submitter may have missed. | ||
| - | - Check to see if [[https://coderoman.wordpress.com/2022/10/11/ | + | - Check to see if [[https://tehguide.com/ |
| - Check for typos in the comments and readmes. | - Check for typos in the comments and readmes. | ||
| - If you’re not proficient with the programming language or framework used, make a note of that in your review. | - If you’re not proficient with the programming language or framework used, make a note of that in your review. | ||
reviewing-a-pull-request.1695224449.txt.gz · Last modified: 2023/09/20 15:40 by Roman Sheydvasser