5 ms·

How do you bootstrap the trust without code review? Always confused how teams without review onboard someone new.

by rgbrgb 2y ago

ajmurmann 2y ago
Pair-programming is one option. It might seem expensive at first glance but usually both parties get a lot of value from it (new possible ideas for the teacher if the learner isn't entirely junior and obviously quick learning for the new person)
- Spivak 2y ago
  We do pair programming for new hires, works well. I'm currently frustrated because our code reviews are both mandatory due to security certifications but also completely worthless to our team structure except Slack spam asking someone to click the approve button. * The work that is done is decided beforehand, the code in every PR corresponds to a card that's already been discussed. * There's no incentive whatsoever to "sneak something in" and if you do so maliciously you'll be fired and maybe prosecuted depending on the damage. * Your code goes through integration testing and QA so you'll never (immediately) take down prod. * I have backups coming out my ears that assume the code running on our app servers is actively malicious so you couldn't cause data loss if you tried. * Norms are all enforced in code, which makes discussions about style pointless. If it passes CI it's good enough.
  - ajmurmann 2y ago
    To play devil's advocate: "* There's no incentive whatsoever to "sneak something in" and if you do so maliciously you'll be fired and maybe prosecuted depending on the damage." How would you discover the snuck in code in a timely fashion?
criley2 2y ago
I think you just accept the risk, same as not doing code review. If you're doing something Serious™ then you follow the serious rules to protect your serious business. If it doesn't actually matter that much if prod goes down or you release a critical vulnerability or leak client data or whatever, then you accept the risk. It's cheaper to push out code quickly and cleanup the mess as long as stakeholders and investors are happy.
deleted 2y ago
[deleted]

How do you bootstrap the trust without code review? Always confused how teams without review onboard someone new.

by rgbrgb 2y ago

How do you bootstrap the trust without code review? Always confused how teams without review onboard someone new.

Pair-programming is one option. It might seem expensive at first glance but usually both parties get a lot of value from it (new possible ideas for the teacher if the learner isn't entirely junior and obviously quick learning for the new person)

Spivak 2y ago
We do pair programming for new hires, works well. I'm currently frustrated because our code reviews are both mandatory due to security certifications but also completely worthless to our team structure except Slack spam asking someone to click the approve button. * The work that is done is decided beforehand, the code in every PR corresponds to a card that's already been discussed. * There's no incentive whatsoever to "sneak something in" and if you do so maliciously you'll be fired and maybe prosecuted depending on the damage. * Your code goes through integration testing and QA so you'll never (immediately) take down prod. * I have backups coming out my ears that assume the code running on our app servers is actively malicious so you couldn't cause data loss if you tried. * Norms are all enforced in code, which makes discussions about style pointless. If it passes CI it's good enough.
- ajmurmann 2y ago
  To play devil's advocate: "* There's no incentive whatsoever to "sneak something in" and if you do so maliciously you'll be fired and maybe prosecuted depending on the damage." How would you discover the snuck in code in a timely fashion?

criley2 2y ago

I think you just accept the risk, same as not doing code review. If you're doing something Serious™ then you follow the serious rules to protect your serious business. If it doesn't actually matter that much if prod goes down or you release a critical vulnerability or leak client data or whatever, then you accept the risk. It's cheaper to push out code quickly and cleanup the mess as long as stakeholders and investors are happy.

deleted 2y ago

[deleted]