Skip to content

Instantly share code, notes, and snippets.

@maxparm

maxparm/pr-assessment.md

Last active October 18, 2016 18:37
Show Gist options
  • Star 2 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save maxparm/47afbf4a54e5be4a680c to your computer and use it in GitHub Desktop.
Save maxparm/47afbf4a54e5be4a680c to your computer and use it in GitHub Desktop.
Download ZIP
Pull Request Assessment
Raw
pr-assessment.md

Table to Predict Difficulty of a PR (yes = 1, no = 0)

  • Is it involving back-end and front-end? 1
  • Is it involving multiple models/apps? 1
  • Is a database schema change required? 1
  • How many ui screens are involved? N (as number of UI screens involved)
  • Is it involving changes on the mobile apps/extensions? N (as number of apps/extensions involved)
  • Is this a new feature? 1 (due to uncertainity)
  • Will a feature switch be used? 1
  • Does it involve old code (> 2 years old)? 1
  • Do we need to upgrade a library? 1
  • Are unit tests needed? 1
  • Are functional tests needed? 1
  • Do we need to add customer facing copy/error messages?
  • Bug: You don't know exactly the cause of the bug?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment