Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Meta: set version for "build-helper-maven-plugin" #89

Merged
merged 2 commits into from
May 17, 2018
Merged

Meta: set version for "build-helper-maven-plugin" #89

merged 2 commits into from
May 17, 2018

Conversation

jmini
Copy link
Member

@jmini jmini commented May 17, 2018

This PR removes those warnings

[INFO] Scanning for projects...
[WARNING] 
[WARNING] Some problems were encountered while building the effective model for org.openapitools:myClientCodegen-openapi-generator:jar:1.0.0
[WARNING] 'build.plugins.plugin.version' for org.codehaus.mojo:build-helper-maven-plugin is missing. @ line 65, column 21
[WARNING] 
[WARNING] It is highly recommended to fix these problems because they threaten the stability of your build.
[WARNING] 
[WARNING] For this reason, future Maven versions might no longer support building such malformed projects.
[WARNING]

That are created when this command is executed:

mvn verify -f samples/meta-codegen/lib/pom.xml

Corresponding script: bin/meta-codegen.sh

@wing328
Copy link
Member

wing328 commented May 17, 2018

Tested locally and confirmed the warning has gone.

@wing328 wing328 added this to the 3.0.0 milestone May 17, 2018
@wing328 wing328 merged commit 2821f18 into OpenAPITools:master May 17, 2018
@jmini jmini deleted the set_build-helper-maven-plugin_version branch May 17, 2018 15:56
jimschubert added a commit to jimschubert/openapi-generator that referenced this pull request May 23, 2018
@jimschubert
Merge branch 'master' into cli-generator-name-option
2a14897 
* master: (36 commits)
  jaxrs-cxf-cdi: fix outer enum (OpenAPITools#131)
  Move bash argv opt to end of ags line in scripts (OpenAPITools#124)
  Reduce CI logging (OpenAPITools#119)
  Download elm dependencies without prompting user. (OpenAPITools#118)
  [aspnetcore] Make the use of Swashbuckle optional (OpenAPITools#110)
  DefaultGenerator: fix NullPointerException (OpenAPITools#109)
  [Java] use html entities in javadoc of generated code (OpenAPITools#106)
  Update PULL_REQUEST_TEMPLATE.md
  [java] Enum in array of array (OpenAPITools#66)
  Rename datatype to dataType in CodegenProperty (OpenAPITools#69)
  update elm test to compile all elm files (OpenAPITools#95)
  Fix Petstore example for Elm (OpenAPITools#96)
  Update Docker documentation (OpenAPITools#97)
  CaseFormatLambda has been added, params for Rest-assured client has been refactored (OpenAPITools#91)
  Update integration.md
  [Clojure] Add util method to set the api-context globally (OpenAPITools#93)
  [JaxRS-Java] issue with implFolder on windows, and required fields generation for containers (OpenAPITools#88)
  Set parameters allowableValues dynamically (OpenAPITools#65)
  Meta: set version for "build-helper-maven-plugin" (OpenAPITools#89)
  Fix javadoc issues in "openapi-generator" module (OpenAPITools#84)
  ...
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@wing328 wing328 wing328 approved these changes

Assignees
No one assigned
Labels
Enhancement: Code format
Projects
None yet
Milestone
3.0.0
Development

Successfully merging this pull request may close these issues.
2 participants
@jmini @wing328