I gave commit rights to someone I didn't know (2016)
I know this is not point of the article but:<p>> The PR was bigger than what I felt I could sensibly review and, in honesty, my desire to go through the hours of work I could tell this would take for a project I no longer used was not stellar.<p>The PR: <a href="https://github.com/django-money/django-money/pull/2/files?diff=split&w=1">https://github.com/django-money/django-money/pull/2/files?di...</a><p>Do others share this sentiment?<p>This doesn't look like a particularly big PR to me, judging solely by the amount of code changed and the nature of the changes at first glance.<p>Are most of your PRs at work tiny, couple lines of code at most? Am I sloppy for not even consider reviewing this for "hours"? Are all code bases I have worked on sloppy because features often require changing more code than this?