DEV Community

loading...

Discussion on: What is your team code review process?

Collapse
perigk profile image
Periklis Gkolias

For all the companies I have worked for it is pretty much identical. A dev branches out from master/main/sprint/develop branch.

They do the work. They open a pull request against the origin branch.

The reviewer checks the diffs (for styling, readability, approach, architecture etc), does a smoke test, maybe uses tools like linters or sonar to point out problems in disguise.

If they exist, checks the unit tests as well (do they succeed, do they need to be amended, do we need more).

Forem Open with the Forem app