Presentations
Templates
Features
Teams
Pricing
Log in
Sign up
Log in
Sign up
Menu
Code Reviews at HackSoft
Team Rules
Click me
Try to keep the PR small
Open PR as soon as possible
Separate refactoring PRs
Basic AF
Smoke testing
Is the name of the PR good?
Is the build passing?
Is there enough context?
If it's a new feature, why does it have deletes?
Example 1
Example 2
If it's changing something, should it be backwards compatible?
Example 1
Example 2
If it's changing something, is there a regression somewhere else?
Example 1
Example 2
If it's an addition to an existing feature, does it look like everything else? Why not?
If it's a refactoring, does it break something? Usually needs a lot of clicking.
Style checks. Lowest of priorities.
My common sense is tingling ...
Q&A
deck
By Hack Bulgaria
Made with Slides.com
deck
1,074
Hack Bulgaria
github.com/HackBulgaria
hackbulgaria.com
hackbulgaria
More from
Hack Bulgaria