Pull Request Etiquette

Marcus Calhoun-Lopez mcalhoun at macports.org
Sat May 19 22:22:16 UTC 2018


Thank you for the advice.

I do not believe I was clear.
The pull request is *not* mine.
I did, however, use it as a staring point for local changes.

Thanks,
Marcus

> On May 19, 2018, at 3:17 PM, Helmut K. C. Tessarek <tessarek at evermeet.cx> wrote:
> 
> Hello,
> 
> On 2018-05-20 00:06, Marcus Calhoun-Lopez wrote:
>> Does anyone have any suggestions on the proper way forward?
> 
> It's a PR, you are the owner. Rebase and force push.
> 
> Unless there's a reason to keep the history there's no reason why not to
> force push.
> 
>> It seems inefficient to attempt to describe the changes in the current pull request comment section.
> 
> It depends. Is there a reason to keep the process of getting where you
> are right now alive?
> 
>> It seems a little silly to open a competing pull request.
> 
> Yes. Unless you close the previous one.
> 
>> Should I create a pull request on the pull request?
> 
> No.
> 
> I'm not a MacPorts dev so my opinion has not much weight, but I do know
> how to work with git and in a team.
> 
> Cheers,
> K. C.
> 
> 
> -- 
> regards Helmut K. C. Tessarek              KeyID 0x172380A011EF4944
> Key fingerprint = 8A55 70C1 BD85 D34E ADBC 386C 1723 80A0 11EF 4944
> 
> /*
>   Thou shalt not follow the NULL pointer for chaos and madness
>   await thee at its end.
> */
> 



More information about the macports-dev mailing list