Progrider

Programming discussion

Search

Searches topics and replies. Every word has to appear in the post.

5 posts found.

PostPosted
General · post #5

For me it explains the compiler errors quite well. And also sometimes confidently wrong. I still ask here when it's important.


by tomasz
General · post #4

Rejecting it is easy, it's the time spent replying politely that adds up. I've started asking "which case did you see fail?". It mostly ends the conversation.


by lintroller !-TjZOnKc
General · post #3

I use one for boilerplate and it's fine. If I couldn't explain the diff I wouldn't send it. It's the not reading it that's the problem, you're right.


by sam
General · post #2

Used to be that bad code at least had the decency to be short.


by vek
General · post #1

Got a review comment this week, four paragraphs, very polite, on a one line change. Told me to consider edge cases in a function that takes a boolean. Then a PR where the description explained a feature the diff didn't contain. I don't mind…


by gcfail
All times shown in UTC.