this post was submitted on 08 Jun 2023
0 points (NaN% liked)

Programming

13367 readers
1 users here now

All things programming and coding related. Subcommunity of Technology.


This community's icon was made by Aaron Schneider, under the CC-BY-NC-SA 4.0 license.

founded 1 year ago
MODERATORS
 

Hey all! My team at work is struggling with growing pains of getting into a formalized review process, so I was wondering if any of you guys have some things to live or die by in your code reviews. How much of it is manual, or how much is just static code analysis + style guide stuff, etc?

top 5 comments
sorted by: hot top controversial new old
[–] DmMacniel@feddit.de 0 points 1 year ago (2 children)

have you looked at Git Flow? Its pretty solid.

My team has a develop branch from which we branch feature branches. On it we commit our stuff and when we think its feature complete we build a snapshot version of it so that our QA can test it. Once that test was successful, and the code has been peer reviewed, it will be merged back onto develop.

PRs will be auto built so that the feature can be integrated and automated tested.

[–] Kaldo@beehaw.org 0 points 1 year ago (1 children)

Do you have automated builds for every snapshot that needs to be tested or is QA running them locally?

[–] DmMacniel@feddit.de 0 points 1 year ago

We have a pipeline (realised on Jenkins with jenkinfiles) that is triggered by changes on a Branch a PR was made for. Another pipeline then is responsible for providing the Application for QA testing.

[–] pattern@beehaw.org 0 points 1 year ago (1 children)

So we use that, and that works well. What does your peer review process look like? Is it pretty loosy-goosy, or do you have hard and fast guidelines and checklists?

[–] DmMacniel@feddit.de 0 points 1 year ago

Its pretty much loosy-goosy with only a limit of how many checks are required (including a fully built build) to be able to merge.