Committers,
Please make sure to follow Apex community guideline when merging PR
http://apex.apache.org/contributing.html.
1. Ensure that basic requirements for a pull request are met. This
includes:
* Sufficient time has passed for others to review
* PR was suffiently reviewed and comments were addressed.
Seevoting policy <https://www.apache.org/foundation/voting.html>.
* When there are multiple reviewers, wait till other reviewers
approve, with timeout of 48 hours before merging
* /If the PR was open for a long time, email dev@ declaring intent
to merge/
* Commit messages and PR title need to reference JIRA (pull
requests will be linked to ticket)
* /Travis CI and Jenkins pull request build needs to pass/
* /Ensure tests are added/modified for new features or fixes/
* Ensure appropriate JavaDoc comments have been added
* Verify contributions don't depend on incompatible licences
(seehttps://www.apache.org/legal/resolved.html#category-x)
2. Use the github/rebase and merge/option or the git command line to
merge the pull request (see link|view command line options|on the PR).
3. /Update JIRA after pushing the changes. Set the////|Fix
version|////field and resolve the JIRA with proper resolution. *Also
verify that other fields (type, priority, assignee) are correct*./
A couple of recent PR merges (#661, #669) to apex-malhar require a
second look from the committers.
Thank you,
Vlad
