search

mulesoft-labs / raml-jaxrs-codegen

Tools to enable RAML-first development in JAX-RS projects
Other
40 stars 44 forks source link

Gradle plugin for RAML JAX-RS code generation #10

Closed jdpgrailsdev closed 3 years ago

jdpgrailsdev commented 10 years ago

This pull request contains a Gradle plugin for RAML JAX-RS code generation. It is modeled after the Maven plugin in terms of configuration options and logic. The included README contains information on usage and configuration. I have also added Gradle build scripts to the Jersey sample application, which can be used to test the plugin. The plugin itself still needs to have a Gradle task added to support the publishing of the built artifact to a remote Maven repository. To try this all out:

It is also worth noting that there is an issue with the Gradle License plugin, so its currently set to non-strict mode and there is a comment in the build.gradle file of the RAML plugin.

jdpgrailsdev commented 10 years ago

Publishing support has been added.

scholarsmate commented 9 years ago

Would be great if this PR could be merged. What's the hold up?

petrochenko-pavel-a commented 9 years ago

Hi, scholarsmate

I'm working on integrating pull requests of the "mulesoft/raml-jaxrs-codegen" repository. I consider your contribution very valuable. In order to work with it, and finally accept it, I would need you to accept the "MuleSoft Contributor Agreement Acceptance" at https://api-notebook.anypoint.mulesoft.com/notebooks#bc1cf75a0284268407e4 as described in the README of the project.

Thanks for contributing. Regards, Pavel.

jdpgrailsdev commented 9 years ago

I have accepted the Contributor Agreement. Let me know if you need anything else.

JohnBaileyN commented 9 years ago

What is the status on getting this merged?

jdpgrailsdev commented 9 years ago

@JohnBaileyN I haven't heard a word from the maintainers of the repo as to the status. The branch most likely needs updating, as I created the plugin over a year ago.

petrochenko-pavel-a commented 9 years ago

Actually it is merged. But not tested really well. So any issues are very appreciated.

Regards, Pavel

On Wed, Apr 15, 2015 at 8:56 PM, Jonathan Pearlin notifications@github.com wrote:

@JohnBaileyN https://github.com/JohnBaileyN I haven't heard a word from the maintainers of the repo as to the status. The branch most likely needs updating, as I created the plugin over a year ago.

— Reply to this email directly or view it on GitHub https://github.com/mulesoft/raml-jaxrs-codegen/pull/10#issuecomment-93433461 .

JohnBaileyN commented 9 years ago

@petrochenko-pavel-a I do not see this as being merged, both in terms of the status of this Pull Request (Open) and in that the directories in the master do not contain the changes.

petrochenko-pavel-a commented 9 years ago

Ah, this is because project now lives at https://github.com/mulesoft/raml-for-jax-rs/ please see https://github.com/mulesoft/raml-for-jax-rs/tree/master/raml-to-jaxrs/gradle-plugin. And thanks again for this contribution.

Regards, Pavel

On Thu, Apr 16, 2015 at 1:31 AM, John Bailey notifications@github.com wrote:

@petrochenko-pavel-a https://github.com/petrochenko-pavel-a I do not see this as being merged, both in terms of the status of this Pull Request (Open) and in that the directories in the master do not contain the changes.

— Reply to this email directly or view it on GitHub https://github.com/mulesoft/raml-jaxrs-codegen/pull/10#issuecomment-93539843 .

jdpgrailsdev commented 9 years ago

@petrochenko-pavel-a Sounds like I should close this PR then, so that it will stop the confusion.

petrochenko-pavel-a commented 9 years ago

Yep, thanks a lot

On Tue, Apr 21, 2015 at 8:55 PM, Jonathan Pearlin notifications@github.com wrote:

@petrochenko-pavel-a https://github.com/petrochenko-pavel-a Sounds like I should close this PR then, so that it will stop the confusion.

— Reply to this email directly or view it on GitHub https://github.com/mulesoft/raml-jaxrs-codegen/pull/10#issuecomment-94830054 .