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? :)
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...