A quick question on PR etiquette

Mark Anderson mark at macports.org
Mon Jan 21 14:11:15 UTC 2019


Cool. That makes a lot of sense,.Thanks!

Mark

On Sun, Jan 20, 2019 at 4:24 PM Mojca Miklavec <mojca at macports.org> wrote:

> On Sun, 20 Jan 2019 at 22:18, Mojca Miklavec wrote:
> > On Sun, 20 Jan 2019 at 19:45, Mark Anderson wrote:
> > >
> > > So, I want to change my email on all of my ports. Should I do them all
> in one big PR which is what my gut says, or should I do a separate one for
> each? It'd be the only change I'd make in this pass.
> >
> > Since you have commit rights: if you are sure that the change is
> > correct and make sure that nothing breaks (due to a typo like
> > forgotten brace etc.), there is no need to make a pull request.
>
> ... but if you do make a pull request, please don't open any more than
> a single one. I initially thought you were asking whether you need to
> make one commit or multiple commits. I would strongly prefer a single
> commit as well. Opening multiple pull request would only add a lot of
> burden to developers.
>
> (What's important is not to make completely unrelated changes to
> multiple ports in one commit or one pull request.)
>
> Mojca
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macports.org/pipermail/macports-dev/attachments/20190121/8aced245/attachment.html>


More information about the macports-dev mailing list