sprintly/sprint.ly-culture — pull request #25 by yathit (2014-07-30 23:49:38+00:00)
Title: MVP PR - Minimum viable patch PR

MVP PR - Minimun viable patch PR

Prefix Pull Request (PR) with MVP to indicate minimum viable patch to reviewers.

When committer is not sure of the patch is appropriate for the fix or because of alternative implementation are possible. MVP PR don't have unit tests and may not fix all possible issues. Mostly it just fix the main issue. Committer should reveal alternative implementations and known issues.

When reviewing MVP PR, reviewer should focus on solution approach not the implementation of PR. Reviewer should see overall problem, alternative solutions, appropriateness of the fix.

Reviewer may not comments on MVP PR codes.