Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[JAXRS-CXF-CDI] add POM template #3958

Merged
merged 5 commits into from
Oct 12, 2016
Merged

Conversation

nickcmaynard
Copy link
Contributor

@nickcmaynard nickcmaynard commented Oct 10, 2016

PR checklist

  • Read the contribution guildelines.
  • Ran the shell/batch script under ./bin/ to update Petstore sample so that CIs can verify the change. (For instance, only need to run ./bin/{LANG}-petstore.sh and ./bin/security/{LANG}-petstore.sh if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates)
  • Filed the PR against the correct branch: master for non-breaking changes and 2.3.0 branch for breaking (non-backward compatible) changes.

Description of the PR

For jaxrs-cxf-cdi, add pom.xml template, fix a bug, and update samples.

@nickcmaynard nickcmaynard changed the title Cxf cdi jaxrs-cxf-cdi POM template Oct 10, 2016
@wing328 wing328 added this to the v2.2.2 milestone Oct 11, 2016
@wing328
Copy link
Contributor

wing328 commented Oct 11, 2016

@nickcmaynard thanks for the PR to add pom.xml. When I ran mvn test, I got some errors:

[ERROR] symbol:   variable ApiResponseMessage
[ERROR] location: class io.swagger.api.impl.PetApiServiceImpl
[ERROR] /Users/williamcheng/Code/swagger-api/swagger-codegen/samples/server/petstore/jaxrs-cxf-cdi/src/main/java/io/swagger/api/impl/PetApiServiceImpl.java:[62,39] cannot find symbol
[ERROR] symbol:   class ApiResponseMessage
[ERROR] location: class io.swagger.api.impl.PetApiServiceImpl
[ERROR] /Users/williamcheng/Code/swagger-api/swagger-codegen/samples/server/petstore/jaxrs-cxf-cdi/src/main/java/io/swagger/api/impl/PetApiServiceImpl.java:[62,58] cannot find symbol
[ERROR] symbol:   variable ApiResponseMessage
[ERROR] location: class io.swagger.api.impl.PetApiServiceImpl
[ERROR] -> [Help 1]
[ERROR] 
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR] 
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException

Did mvn test pass locally in your environment?

@nickcmaynard
Copy link
Contributor Author

@wing328 That should fix it. Apologies - didn't spot a skipped overwrite when regenerating the samples.

@wing328
Copy link
Contributor

wing328 commented Oct 12, 2016

@nickcmaynard thanks for the quick fix. mvn test passed locally for me.

@wing328 wing328 merged commit 823ce72 into swagger-api:master Oct 12, 2016
acramatte added a commit to comerge/swagger-codegen that referenced this pull request Oct 12, 2016
* upstream/master:
  [html]Group api index by operations.baseName (swagger-api#3953)
  Revert "[WIP] Improve PHP client emitted code quality"
  update retrofit1,2 samples
  jaxrs-cxf-cdi POM template (swagger-api#3958)
  [Swift] Add / as enum separator
  issue-890 correct fix for deprecated Jersey method
  2208 fix Objc Mapping Generation (swagger-api#3894)
  [Spring] Format datetime in rfc3339 (swagger-api#3777)
  [Java/Jackson] use a jdk6 compatible DateFormat for java.util.Date (swagger-api#3768)
  add template owner jax-rs cxf cdi
  remove newline char in *.mustache; added generated code
  Add a new JAX-RS server generator - jaxrs-cxf-cdi (swagger-api#3940)
  added package paths for retrofit class names
  feature(PHP QA) add initial PHP client template tweaks to improve emitted code quality
  [resteasy] configure jackson to use rfc3339 dates
  [jersey] configure jackson to use rfc3339 dates
@wing328 wing328 changed the title jaxrs-cxf-cdi POM template [JAXRS-CXF-CDI] add POM template Feb 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants