Ok, lemme do both.
Thanks for the quick feedback
--
Jean-Louis Monteiro
http://twitter.com/jlouismonteiro
http://www.tomitribe.com


On Tue, Nov 24, 2020 at 4:43 PM Romain Manni-Bucau <[email protected]>
wrote:

> Hi JL,
>
> we don't use travis so guess it never worked but fine for me to:
>
> 1. merge (since build passed - + I validated this one locally and discussed
> it with you ;))
> 2. drop enforcer plugin
>
> Romain Manni-Bucau
> @rmannibucau <https://twitter.com/rmannibucau> |  Blog
> <https://rmannibucau.metawerx.net/> | Old Blog
> <http://rmannibucau.wordpress.com> | Github <
> https://github.com/rmannibucau> |
> LinkedIn <https://www.linkedin.com/in/rmannibucau> | Book
> <
> https://www.packtpub.com/application-development/java-ee-8-high-performance
> >
>
>
> Le mar. 24 nov. 2020 à 16:38, Jean-Louis MONTEIRO <[email protected]> a
> écrit :
>
> > Hi,
> >
> > I pushed a PR which looks good as per feedback on review
> > https://github.com/apache/johnzon/pull/68
> >
> > The build looks ok on JDK8 but Travis seems unhappy with JDK11
> > It fails with the enforcer which seems relevant. Not sure why we have a
> > JDK11 build if the enforcer is configured to not allow it.
> >
> > What's your preference?
> > Fix Travis build to not build JDK11?
> > Fix Johnzon to allow JDK11 builds?
> >
> > --
> > Jean-Louis
> >
>

Reply via email to