Am 2019-01-06 um 20:41 schrieb Sylwester Lachiewicz:
Hi, a small reminder - anyone to second MNG-5995 and MNG-6069?
I'd like to include both in next release.

I am fine with both, but for some comments on the MNG-6069 PR.

Michael

pt., 30 lis 2018 o 00:21 Sylwester Lachiewicz <slachiew...@gmail.com>
napisał(a):

Hi,
branches for MNG-5995 and MNG-6069 ready in maven core - build passed


https://builds.apache.org/view/M-R/view/Maven/job/maven-box/job/maven/job/MNG-5995/

https://builds.apache.org/view/M-R/view/Maven/job/maven-box/job/maven/job/MNG-6069v2/

Please review.

BR
Sylwester

czw., 8 lis 2018 o 20:52 Karl Heinz Marbaise <khmarba...@gmx.de>
napisał(a):

Hi,

On 08/11/18 20:33, Sylwester Lachiewicz wrote:
Hi Karl, today i removed my branches with fixes and proposals from
gitbox
and leave only PR in Github for easier commenting.

As I mentioned already ...create the branches on maven-core (gitbox
url). The branches will be mirrored to github as well...otherwise we
have no check with our CI which is vital...

Apart from that you can comment the commit on branches in GitHub...we
don't loose anything (maybe some convenience..)...But it might be good
thing to check with INFRA if that can be improved more ...

Kind regards
Karl Heinz Marbaise


Sylwester

czw., 8 lis 2018 o 20:18 użytkownik Karl Heinz Marbaise <
khmarba...@gmx.de>
napisał:

Hi Sylwester,

On 08/11/18 20:00, Sylwester Lachiewicz wrote:
Hi, I have 2 fixes (leftovers from 3.6.0) ready to merge - anyone to
review?

[MNG-6069] https://github.com/apache/maven/pull/153

I can't see the code in the branch MNG-6069 ...(Ok..was originally mine
but you can simply push (force) that code on that branch)..... hijack
that branch ...

Furthermore if I check the old branch (If this is the correct one):


https://builds.apache.org/view/M-R/view/Maven/job/maven-box/job/maven/job/slach-build8/

That contains test failures...

So -1 to merge...


[MNG-5995] https://github.com/apache/maven/pull/185

Unfortunately I don't see a branch here as well which has been built on
CI ...

https://builds.apache.org/view/M-R/view/Maven/job/maven-box/job/maven/
whch job?

It would be great if you could create an appropriate branch on Maven
Core named by the issue which is built on CI ...

so I have to say -1 here as well...


Kind regards
Karl Heinz Marbaise


Sylwester









---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org

Reply via email to