• 0 Posts
  • 5 Comments
Joined 1 year ago
cake
Cake day: June 9th, 2023

help-circle



  • We’re a small team of 6 and follow Git Flow for branching. Every change that wants to go into the develop branch needs an approval on the pull request by one of the two senior developers on the team.

    We have (virtual) meetings for any changes that require discussion; some things are difficult to quickly communicate over pull request comments.

    When it comes to code style, our standard is “format it using IntelliJ”. Same with warnings – if the IDE calls it out, it’s fair game in the review. Personally, I check out every branch I’m reviewing so that I can navigate easily and enable my own warnings, which are generally strict.

    Most PR comments start as questions because sometimes questionable looking code can be correct, then things go from there.