SGTM.
On Fri, Jul 14, 2017 at 3:37 PM, Kenneth Knowles
wrote:
> I think 1 & 3 can be squashed :-)
>
> If someone has fixup! commits you definitely don't want to merge exactly
> as-is so it shouldn't be possible. (and if they messed up their fixup! so
> it doesn't
I think 1 & 3 can be squashed :-)
If someone has fixup! commits you definitely don't want to merge exactly
as-is so it shouldn't be possible. (and if they messed up their fixup! so
it doesn't vanish, you also don't want to merge that)
And then, yes, that covers my wants.
On Fri, Jul 14, 2017 at
Hey folks, thanks for your thoughts!
I have some responses for you. :)
Regarding squashing, it looks like there are three things at play --
forgive me if I've misunderstood.
1. Merge commits exactly as they are in the PR.
2. Squash all commits down to the first commit
3. Automatically squash
On Wed, Jul 12, 2017 at 12:08 PM, Robert Bradshaw <
rober...@google.com.invalid> wrote:
> On Tue, Jul 11, 2017 at 7:14 PM, Kenneth Knowles
> wrote:
>
> > The thing is that "fixup! " indicates that this fixup
> > should be reordered and applied to the referenced commit.
On Tue, Jul 11, 2017 at 7:14 PM, Kenneth Knowles
wrote:
>
> On Tue, Jul 11, 2017 at 4:25 PM, Robert Bradshaw <
> rober...@google.com.invalid> wrote:
>
> > On Tue, Jul 11, 2017 at 8:51 AM, Kenneth Knowles
> > wrote:
> > > I like the idea of
On Tue, Jul 11, 2017 at 4:25 PM, Robert Bradshaw <
rober...@google.com.invalid> wrote:
> On Tue, Jul 11, 2017 at 8:51 AM, Kenneth Knowles
> wrote:
> > I like the idea of controlling squashing or not explicitly in the
> mergebot
> > invocation. I don't think it needs to
I like the idea of controlling squashing or not explicitly in the mergebot
invocation. I don't think it needs to be made interactive, but just based
on preparing the PR appropriately.
I propose this for the default `@asfgit merge`: Don't squash, but reject
merges that have commits that are
(quick update re #2 above): ~4 minutes after I reopened the ticket, it's
fixed.
https://github.com/apache/infrastructure-puppet/commit/709944291da5e8aea711cb8578f0594deb45e222
updates the website to the correct address. Infra is once again the best.
On Mon, Jul 10, 2017 at 12:38 PM, Jason Kuster
Glad to hear everyone's pretty happy about it! Have a couple answers for
your questions.
Ted: I believe the MFA stuff (two-factor auth on github) is necessary for
getting the additional features on GitHub (reviewer, etc), but may not be
necessary for MergeBot. I'll check in with Infra and get
+1 Awesome work!
Thank you Jason!!!
Mark
On Mon, Jul 10, 2017 at 10:05 AM, Robert Bradshaw <
rober...@google.com.invalid> wrote:
> +1, this is great! I'll second Ismaël's list requests, especially 1 and 3.
>
> On Mon, Jul 10, 2017 at 2:09 AM, Ismaël Mejía wrote:
> >
This is really good!!
Regards,
Tarush
On Sat, 8 Jul 2017 at 10:20 AM, Jean-Baptiste Onofré
wrote:
> That's awesome !
>
> Thanks Jason !
>
> Regards
> JB
>
> On 07/07/2017 10:21 PM, Jason Kuster wrote:
> > Hi Beam Community,
> >
> > Early on in the project, we had a number of
This is awesome!!
On Fri, Jul 7, 2017 at 1:46 PM, Thomas Groh
wrote:
> Super duper cool. Very exciting.
>
> On Fri, Jul 7, 2017 at 1:40 PM, Ted Yu wrote:
>
> > For https://gitbox.apache.org/setup/ , after completing the first two
> > steps, is
Super duper cool. Very exciting.
On Fri, Jul 7, 2017 at 1:40 PM, Ted Yu wrote:
> For https://gitbox.apache.org/setup/ , after completing the first two
> steps, is there any action needed for "MFA Status" box ?
>
> Cheers
>
> On Fri, Jul 7, 2017 at 1:37 PM, Lukasz Cwik
For https://gitbox.apache.org/setup/ , after completing the first two
steps, is there any action needed for "MFA Status" box ?
Cheers
On Fri, Jul 7, 2017 at 1:37 PM, Lukasz Cwik
wrote:
> for i in range(0, inf): +1
>
> Note that the URL for gitbox linking is:
>
for i in range(0, inf): +1
Note that the URL for gitbox linking is:
https://gitbox.apache.org/setup/ (above
URL was missing '/' and was giving 404)
On Fri, Jul 7, 2017 at 1:21 PM, Jason Kuster wrote:
> Hi Beam Community,
>
> Early on in the project, we had a
This is fantastic!
Thank you Jason, and special thanks to the Infrastructure team for
supporting us every step of the way.
On Fri, Jul 7, 2017 at 1:21 PM, Jason Kuster wrote:
> Hi Beam Community,
>
> Early on in the project, we had a number of discussions about
Hi Beam Community,
Early on in the project, we had a number of discussions about creating an
automated tool for merging pull requests. I’m happy to announce that we’ve
developed such a tool and it is ready for experimental usage in Beam!
The tool, MergeBot, works in conjunction with ASF’s
17 matches
Mail list logo