-
Notifications
You must be signed in to change notification settings - Fork 388
Code Review Checklist
ajayhn edited this page Jan 26, 2016
·
2 revisions
- Is there any upgrade implication? If so at a minimum need
- tag on bug
- tools to address it
- documentation on impact (tag it as release-note in bug)
- Is there any change in schema? If so
- make sure fields aren't removed (can comment as deprecated)
- make sure fields are defined at end (to ensure that if constructors were called without named parameters old code will still work)