1 point
@Vordimous @linuxgal when I see this sort of thing when reviewing PRs, I write words to the effect of “tell me WHY it’s doing that, not WHAT it’s doing”. The “what” is usually obvious from the code and thus not worthy of comment. The “why” will enlighten your colleagues (and future you).
2 points
When you ask a Dev to test their own code
2 points
*
Removed by mod
1 point
Deleted by creator
1 point
But it’s also important to learn that comments should be brief and concise. We have one file from an ex-dev in which there are 750 lines of code and 2000 lines of comment, when someone wants to maintain this code they always have a hard time because this many comments are taking up so much screen real estate that you can’t find the code that actually does stuff
2 points
1 point
