Pull request comments

Terms

• MergeRequest or MR or PullRequest or PR- запрос на слияние веток
• Thread- открытое обсуждение в рамках некоторого участка кода
• AuthorPR - автор PullRequest
• Reviewer - автор Thread'а / участник Thread'а (может быть несколько)

Comment classification

• Major

  • AuthorPR не может закрыть Thread
  • Reviewer будет против, если Thread будет закрыт без изменений
  • При конфликте взглядов привлекаются другие разработчики к обсуждению
  • Важно: Комментарий, оставленный без указания классификатора, имеет классификатор Major

• Minor

  • AuthorPR может закрыть Thread
  • Reviewer не будет против, если Thread будет закрыт без изменений
  • Reviewer может потенциально расстроиться, если код не будет изменен

• SuperMinor

  • AuthorPR может закрыть Thread
  • Reviewer не будет против, если Thread будет закрыт без изменений
  • Reviewer не расстроится, если код не будет изменен

• FYI

  • AuthorPR может закрыть Thread
  • С точки зрения Reviewer'а для AuthorPR'а будет полезно взять на заметку соображения из комментария и использовать их в будущем
  • Комментарий не требует изменений в коде

• Question

  • AuthorPR не может закрыть Thread
  • Reviewer закроет Thread, когда получит достаточно информации по комментарию
  • Комментарий не требует изменений в коде

• Compliment

  • AuthorPR может закрыть Thread
  • Комментарий не требует изменений в коде
  • Комментарий используется в качестве похвалы в рамках измененного участка кода