Skip to content

Instantly share code, notes, and snippets.

Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save dgoings/ecfe4877b5992190e4fc8893c77a7ae6 to your computer and use it in GitHub Desktop.
Save dgoings/ecfe4877b5992190e4fc8893c77a7ae6 to your computer and use it in GitHub Desktop.
Mark pull request comments with a purpose

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

via Scaling Square Register on objc.io

  • Required: this must be fixed before merge
  • Nice to have: this should be fixed eventually
  • A personal preference: I would do this, but you don’t have to.
  • A question: What does this do?
  • A comment: Just pointing something out, no need for reply/action
  • A change: I have pushed a set of changes that I want to call out
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment