Hey Gabriel, great article. I'm curious about the following statement:
Code submitted for PR should be tested by another developer before you open a PR
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!
Are you sure you want to hide this comment? It will become hidden in your post, but will still be visible via the comment's permalink.
Hide child comments as well
Confirm
For further actions, you may consider blocking this person and/or reporting abuse
We're a place where coders share, stay up-to-date and grow their careers.
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!