Re: [isis] fixed broken link (#1)
Dan Haywood <dan@...
2014-09-15 19:34:30 GMT
Per the email train below, Jeroen noticed I merged in a PR from github (a
tiny one line fix to a README, but a PR nevertheless) and wanted to check
that it was ok to do so.
Anyway, just to confirm, it IS ok to merge in PRs, but do see  for more
info on what the great and the good at ASF have to say on the matter.
On 12 September 2014 14:03, Dan Haywood <dan@...>
> No, I did that. I set up a remote to his fork, pulled down his commit,
> and merged it in. That way github automatically closes the pull request.
> My understanding (I hope I'm right on this) is that there's an implicit
> ICLA for anyone raising a PR on an Apache project that they grant the
> rights to ASF for the change.
> But mostly just trying to keep things nice and clean...
> On 12 September 2014 14:00, Jeroen van der Wal <jeroen@...>