|
|
|
|
|
by depaya
1505 days ago
|
|
Yeah I don't agree either. If I'm getting some feedback from a peer and they tell me "Usually your code is extremely well documented and ready to go as is, but your most recent PR doesn't cut it because xyz" I am glad they have included the "but" in the sentence so I have more context for their comment. It accomplishes two things: It lets me know they are providing this feedback from a place where they are familiar with my work (context for the feedback), and it is a complement (which is always nice to receive). True, the actual message is what comes after the "but," but(!) that doesn't make the first half of the sentence unimportant. |
|
"Usually your code is extremely well documented and ready to go as is and I think your most recent PR could be improved in this area." It softens the blow while still retaining the relevant context.