Re: TOMEE-2289

2019-01-04 Thread Romain Manni-Bucau
>> regarding >>> > the >>> > > > > spec which I've asked on the github thread, can you please >>> clarify >>> > > these? >>> > > > > >>> > > > > On Wed 2 Jan, 2019, 6:09 PM Ivan Junckes Filho < >>>

Re: TOMEE-2289

2019-01-04 Thread Ivan Junckes Filho
@gmail.com >> > > > > wrote: >> > > > > >> > > > > > Puneeth, I did some comments here >> > > > > > >> > https://github.com/apache/tomee/pull/340#pullrequestreview-188661807 >> > > , >> > > > > > please take a look. >> > > > > > >> > > > > > On Wed, Jan 2, 2019 at 9:51 AM Ivan Junckes Filho < >> > > > ivanjunc...@gmail.com >> > > > > > >> > > > > > wrote: >> > > > > > >> > > > > > > Hi Puneeth, I will review it for you. >> > > > > > > >> > > > > > > On Wed, Jan 2, 2019 at 8:39 AM Puneeth PS < >> puneeth...@gmail.com> >> > > > > wrote: >> > > > > > > >> > > > > > >> Hi, >> > > > > > >> >> > > > > > >> I have created a PR ( >> https://github.com/apache/tomee/pull/340 >> > ) >> > > > for >> > > > > > >> TOMEE-2289. Can someone take a look and help me improve it? >> > > > > > >> >> > > > > > > >> > > > > > >> > > > > >> > > > >> > > >> > >> >> >> -- >> Atentamente: >> César Hernández Mendoza. >> >

Re: TOMEE-2289

2019-01-03 Thread Ivan Junckes Filho
t; > ivanjunc...@gmail.com > > > > > wrote: > > > > > > > > > > > Puneeth, I did some comments here > > > > > > > > https://github.com/apache/tomee/pull/340#pullrequestreview-188661807 > > > ,

Re: TOMEE-2289

2019-01-03 Thread César Hernández Mendoza
gt; https://github.com/apache/tomee/pull/340#pullrequestreview-188661807 > > , > > > > > please take a look. > > > > > > > > > > On Wed, Jan 2, 2019 at 9:51 AM Ivan Junckes Filho < > > > ivanjunc...@gmail.com > > > > > > > > > > wrote: > > > > > > > > > > > Hi Puneeth, I will review it for you. > > > > > > > > > > > > On Wed, Jan 2, 2019 at 8:39 AM Puneeth PS > > > > wrote: > > > > > > > > > > > >> Hi, > > > > > >> > > > > > >> I have created a PR ( https://github.com/apache/tomee/pull/340 > ) > > > for > > > > > >> TOMEE-2289. Can someone take a look and help me improve it? > > > > > >> > > > > > > > > > > > > > > > > > > > > > -- Atentamente: César Hernández Mendoza.

Re: TOMEE-2289

2019-01-03 Thread Ivan Junckes Filho
Filho < > > ivanjunc...@gmail.com > > > > > > > > wrote: > > > > > > > > > Hi Puneeth, I will review it for you. > > > > > > > > > > On Wed, Jan 2, 2019 at 8:39 AM Puneeth PS > > > wrote: > > > > > > > > > >> Hi, > > > > >> > > > > >> I have created a PR ( https://github.com/apache/tomee/pull/340 ) > > for > > > > >> TOMEE-2289. Can someone take a look and help me improve it? > > > > >> > > > > > > > > > > > > > > >

Re: TOMEE-2289

2019-01-03 Thread Puneeth PS
Filho < > ivanjunc...@gmail.com > > > > > > wrote: > > > > > > > Hi Puneeth, I will review it for you. > > > > > > > > On Wed, Jan 2, 2019 at 8:39 AM Puneeth PS > > wrote: > > > > > > > >> Hi, > > > >> > > > >> I have created a PR ( https://github.com/apache/tomee/pull/340 ) > for > > > >> TOMEE-2289. Can someone take a look and help me improve it? > > > >> > > > > > > > > > >

Re: TOMEE-2289

2019-01-03 Thread Ivan Junckes Filho
Ivan Junckes Filho > > > wrote: > > > > > Hi Puneeth, I will review it for you. > > > > > > On Wed, Jan 2, 2019 at 8:39 AM Puneeth PS > wrote: > > > > > >> Hi, > > >> > > >> I have created a PR ( https://github.com/apache/tomee/pull/340 ) for > > >> TOMEE-2289. Can someone take a look and help me improve it? > > >> > > > > > >

Re: TOMEE-2289

2019-01-02 Thread Puneeth PS
t;> I have created a PR ( https://github.com/apache/tomee/pull/340 ) for > >> TOMEE-2289. Can someone take a look and help me improve it? > >> > > >

[GitHub] tomee issue #340: TOMEE-2289 microprofile openAPI example

2019-01-02 Thread puneethps
Github user puneethps commented on the issue: https://github.com/apache/tomee/pull/340 I noticed one more difference from the spec and the implementation, what I understood from the spec was that there is supposed to be a components field in the yaml or the json if there is a schema

[GitHub] tomee pull request #340: TOMEE-2289 microprofile openAPI example

2019-01-02 Thread puneethps
Github user puneethps commented on a diff in the pull request: https://github.com/apache/tomee/pull/340#discussion_r244777643 --- Diff: examples/mp-openapi/README.adoc --- @@ -0,0 +1,112 @@ +:index-group: MicroProfile +:jbake-type: page +:jbake-status: unpublished +

Re: TOMEE-2289

2019-01-02 Thread Ivan Junckes Filho
t;> Hi, >> >> I have created a PR ( https://github.com/apache/tomee/pull/340 ) for >> TOMEE-2289. Can someone take a look and help me improve it? >> >

[GitHub] tomee pull request #340: TOMEE-2289 microprofile openAPI example

2019-01-02 Thread ivanjunckes
Github user ivanjunckes commented on a diff in the pull request: https://github.com/apache/tomee/pull/340#discussion_r244724535 --- Diff: examples/mp-openapi/README.adoc --- @@ -0,0 +1,112 @@ +:index-group: MicroProfile +:jbake-type: page +:jbake-status: unpublished

[GitHub] tomee pull request #340: TOMEE-2289 microprofile openAPI example

2019-01-02 Thread ivanjunckes
Github user ivanjunckes commented on a diff in the pull request: https://github.com/apache/tomee/pull/340#discussion_r244725387 --- Diff: examples/mp-openapi/src/main/java/org/superbiz/openapi/WeatherService.java --- @@ -0,0 +1,52 @@ +package org.superbiz.openapi; + +

[GitHub] tomee pull request #340: TOMEE-2289 microprofile openAPI example

2019-01-02 Thread ivanjunckes
Github user ivanjunckes commented on a diff in the pull request: https://github.com/apache/tomee/pull/340#discussion_r244725773 --- Diff: examples/mp-openapi/src/main/java/org/superbiz/openapi/WeatherService.java --- @@ -0,0 +1,52 @@ +package org.superbiz.openapi; + +

[GitHub] tomee pull request #340: TOMEE-2289 microprofile openAPI example

2019-01-02 Thread ivanjunckes
Github user ivanjunckes commented on a diff in the pull request: https://github.com/apache/tomee/pull/340#discussion_r244725584 --- Diff: examples/mp-openapi/src/main/java/org/superbiz/openapi/Weather.java --- @@ -0,0 +1,25 @@ +package org.superbiz.openapi; + +import

[GitHub] tomee pull request #340: TOMEE-2289 microprofile openAPI example

2019-01-02 Thread ivanjunckes
Github user ivanjunckes commented on a diff in the pull request: https://github.com/apache/tomee/pull/340#discussion_r244723832 --- Diff: examples/mp-openapi/src/main/java/org/superbiz/openapi/WeatherService.java --- @@ -0,0 +1,52 @@ +package org.superbiz.openapi; + +

[GitHub] tomee pull request #340: TOMEE-2289 microprofile openAPI example

2019-01-02 Thread ivanjunckes
Github user ivanjunckes commented on a diff in the pull request: https://github.com/apache/tomee/pull/340#discussion_r244724385 --- Diff: examples/mp-openapi/README.adoc --- @@ -0,0 +1,112 @@ +:index-group: MicroProfile +:jbake-type: page +:jbake-status: unpublished

Re: TOMEE-2289

2019-01-02 Thread Ivan Junckes Filho
Hi Puneeth, I will review it for you. On Wed, Jan 2, 2019 at 8:39 AM Puneeth PS wrote: > Hi, > > I have created a PR ( https://github.com/apache/tomee/pull/340 ) for > TOMEE-2289. Can someone take a look and help me improve it? >

TOMEE-2289

2019-01-02 Thread Puneeth PS
Hi, I have created a PR ( https://github.com/apache/tomee/pull/340 ) for TOMEE-2289. Can someone take a look and help me improve it?

[GitHub] tomee pull request #340: TOMEE-2289 microprofile openAPI example

2019-01-01 Thread puneethps
GitHub user puneethps opened a pull request: https://github.com/apache/tomee/pull/340 TOMEE-2289 microprofile openAPI example The JIRA status for TOMEE-2289 hasn't changed for quite sometime so I felt I could add a new project for this. Can someone please review the PR