Comment on Python Performance: Why 'if not list' is 2x Faster Than Using len()
gigachad@sh.itjust.works 3 days agoSir, I will make sure to never bother you with a PR and my terrible, terrible code ;)
Comment on Python Performance: Why 'if not list' is 2x Faster Than Using len()
gigachad@sh.itjust.works 3 days agoSir, I will make sure to never bother you with a PR and my terrible, terrible code ;)
sugar_in_your_tea@sh.itjust.works 3 days ago
Rejecting a PR shouldn’t be offensive, it should be a learning opportunity, both for the reviewer and the submitter. If I reject it, I’ll give a clear reason why, and suggestions on how to fix it.
So please bother me with terrible, terrible code. I want to take time out of my day to help contributors learn, and I like pointing out areas where I learn something as well (like, “hey, this is really clever and also really easy to read, good job!”). I’m not always right, but I do have a lot of experience that I think others could benefit from. I know I was deeply appreciative of constructive criticism as a new dev, and I hope that’s true for the people I provide reviews for.