[prev in list] [next in list] [prev in thread] [next in thread] 

List:       macports-dev
Subject:    Re: [macports-ports] branch master updated: remove unused patches
From:       Zero King <l2dy () macports ! org>
Date:       2018-02-12 15:31:00
Message-ID: 20180212153100.yg2lzzqamyu6keyn () macports ! org
[Download RAW message or body]


On Mon, Feb 12, 2018 at 08:30:58AM -0500, Perry E. Metzger wrote:
>On Mon, 12 Feb 2018 03:52:59 -0600 Ryan Schmidt
><ryandesign@macports.org> wrote:
>>
>> Perry, it's your responsibility to ensure commit messages conform
>> to our guidelines before merging a PR.
>>
>
>Gotcha. It's a bit irritating that there isn't a good way to edit a
>commit message in the course of merging, because it seems unfriendly
>to me to make people go around again for something trivial I should
>be able to fix. Anyone know if there's some way to do that?

If the PR author allowed edits from maintainers (on by default), you can
force push to the PR source branch. See
https://trac.macports.org/wiki/WorkingWithGit#WorkingwithsomeoneelsespullrequestthroughitsID
and https://gist.github.com/l2dy/7da9621954ebcf1a19869f391662a41e for
some instructions.

-- 
Best regards,
Zero King

["smime.p7s" (application/x-pkcs7-signature)]

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic