News for hackers. Stuff that matters.
Re:Ship / Show / Ask: A modern branching strategy by lowercased on Monday September 13, 2021 @11:56AM (#28513083)
My code reviews of others is often "please add a test", then nothing happens for 3 days, and I get people saying "when will this be done? it already passed UAT! get it moving". then stuff gets merged without tests, and the cycle repeats. Not ALWAYS, and commit/merge absent tests is sometimes required.

The shoe was on the other foot a few years back and I was contracted on part of a large/legacy enterprise app product. I could 'fix' a bug, but couldn't always write a test, as I just didn't understand the system well enough to grok what was there. Policy was "you have to have a test". Fair enough. I remember hitting up 5-6 other devs on the team - some of whom had been there for 10 years - to pair for an hour or so to help me write a test. The same people who were blocking the PR requiring a test would not help me write a test. Some were honest enough to say "I don't know how you'd test that". But... it was frustrating. I had a 20 minute code patch (which demonstrably fixed the bug to the PM's satisfaction) take 8 calendar days to figure out how to test. Then a reviewer didn't like how the test setup was done. When they learned it was done with code from a colleague (and not of my own doing) they suddenly liked it and said it was 'clever'.

Happy happy days...

Re:Ship / Show / Ask: A modern branching strategy by mixmastamyk on Monday September 13, 2021 @04:23PM (#28516378)
Contractors are generally paid by the hour. So while frustrating, there's always the paycheck to look forward to.
Re:Ship / Show / Ask: A modern branching strategy by ryan_lane on Monday September 13, 2021 @09:39PM (#28519167)
This is why having tests shouldn't be enforced by review, but if the code coverage drops for the PR, it fails tests.
Re:Ship / Show / Ask: A modern branching strategy by dopidopHN on Monday September 13, 2021 @03:52PM (#28515946)
I think I left your previous work place a few month ago. It’s just too similar. Each point.
Re:Ship / Show / Ask: A modern branching strategy by lowercased on Monday September 13, 2021 @04:12PM (#28516212)
was the company in the southeast US? :)