There is no difference whether pull request is created by committer or non committer, both need to create fork as described in http://apex.incubator.apache.org/contributing.html and squash commits as described in item #7. Only committer is able to merge from the fork to the ASF master repository and git will preserve the author (or authors if there are multiple commits from different authors).

The recommendation to squash commit is not much different from other ASF projects commit policy where contributor submits single patch that is later merged by a committer. Committer is not responsible for squashing commits, this should be done by the contributor.

Thank you,

Vlad

On 11/2/15 14:19, Justin Mclean wrote:
Hi,

You may want to be careful if changes were made by multiple authors or on 
behalf of someone else (i.e. pull request by a non committer), especially if 
any of them have not signed an ICLA. IP provenance needs to be preservered. In 
those cases I’d suggest not squashing commits.

Justin

Reply via email to