2 ms·
PR I sent yesterday, line changes: +2,703 −3,529. That's like 50 bucks just for that PR.
by m0zg 7y ago
PR I sent yesterday, line changes: +2,703 −3,529. That's like 50 bucks just for that PR.
- aledalgrande 7y agodude I hope that was the result of updating yarn.lock or similar, otherwise good luck to your reviewer!
- m0zg 7y agoNope, it's mostly code. It's a gnarly PR that atomically delivers a feature (and removes the feature that the new one replaces), but this looks about right for my weekly productivity overall, except I usually submit it in smaller PRs, and the delta is mostly lines added.
- perlgeek 7y agoIf another developer reviewed your code, how much time it would it take them, and how much is that time worth? If you divide the 50 bucks by that number, you get a cost ratio. If it's lower than the ratio of (benefit expected by automatic code review) / (benefit expected by manual code review), it's worth using. I guess we can speculate all we want; in the end, only experience will show if the service is worth it or not.