forked from angular/angular.js
-
Notifications
You must be signed in to change notification settings - Fork 10
Contribution Checklist
petebacondarwin edited this page Feb 4, 2013
·
1 revision
- Feature improves existing core functionality
- API is compatible with existing Angular apis and relevant standards (if applicable)
- Contributor signed CLA now or in the past (after signing leave a comment here with your real name)
- PR contains unit tests
- PR contains e2e tests (if suitable)
- PR contains documentation update
- PR was tested on ci.angularjs.org for cross-browser compatibility
- PR is rebased against recent master
- PR is squashed into one commit per logical change
- PR's commit message is descriptive and allows us to autogenerate release notes (required commit message format)