cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Michal Mocny <>
Subject Re: Unknown "committer"
Date Thu, 29 May 2014 00:12:38 GMT
I think that may be the result of the way we do PR..  I merged the PR Joe
points to, and I did it by pulling from his fork, and merging it into
cordova-android.  In theory, this way any edits I need to do end up in my
merge commit, and the contributor changes are isolated in the original work
without my touching it.

The problem is that in this case a fast-forward was enough, and no merge
commit was needed.  There is a way to merge forcing a merge commit even
when a fast-forward is possible (--no-ff), and I've started using this more
recently.  Probably we should update the docs to add this.

On Wed, May 28, 2014 at 7:55 PM, Joe Bowser <> wrote:

> I'm pretty sure this is more of a git issue than anything else:
> I found an unknown committer in Cordova-Android:
> It would be good to know how this commit wound up in here, since I
> would rather blame a bad git merge.  I'm not going to roll anything
> back because of this trivial commit.
> On Wed, May 28, 2014 at 4:20 PM, Shazron <> wrote:
> >
> >
> > Rowell Cruz, from BlackBerry. He has a listed CLA but is definitely not
> an
> > Apache Cordova committer. Can someone at Blackberry shed some light on
> this?
> >
> > I was triaging issues and came to this issue:
> >
> > and wanted to see who pulled this in, but both author and committer are
> the
> > same. The committer field should be only a Cordova PMC voted committer.

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message