Skip to content

Instantly share code, notes, and snippets.

View erickrawczyk's full-sized avatar

Eric Krawczyk erickrawczyk

View GitHub Profile
@erickrawczyk
erickrawczyk / pr-comments.md
Last active May 12, 2023 13:25 — forked from millerized/comments.md
PR Comments with a Purpose

This is a set of ‘statuses’ that pull request comments must be marked with to help explain intent.

  • Required: this must be fixed before merge
  • Nice to have: this should be fixed eventually
  • Personal Preference: I would do this, but you don’t have to.
  • Question: What does this do?
  • Suggested Change: I have pushed a set of changes that I want to call out*

* not in original article