If you do code review, you likely come across the same issues over and over again. People who are contributing to your code base for the first tim...
For further actions, you may consider blocking this person and/or reporting abuse
Great article! I've been keeping a list of the comments made on pull requests I've opened and have encouraged my coworkers to do the same but I really like the idea of a shared document for the team to reference.
Hey Gabriel, great article. I'm curious about the following statement:
How do you do that in practice?
Hey Sten, bit of a typo on my part, it should read "Code submitted as a PR should be tested by another developer before it's merged"
I'll update the article.
Ah thanks, I thought you were doing some Gerrit-style magic 😅
I've used phabricator in the past to do "non pr" diffs...but yeah, in this case just a typo!