Skip to content

Fix maven plugin NullPointer exception while trying to generate Spring server (fixes #4625)#4626

Merged
wing328 merged 1 commit intoswagger-api:masterfrom
eblis:maven-plugin-npe
Jan 27, 2017
Merged

Fix maven plugin NullPointer exception while trying to generate Spring server (fixes #4625)#4626
wing328 merged 1 commit intoswagger-api:masterfrom
eblis:maven-plugin-npe

Conversation

@eblis
Copy link
Copy Markdown
Contributor

@eblis eblis commented Jan 23, 2017

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

#4625 - Check if configOptions are set before trying to get the source folder, otherwise assume null/empty.

…t the source folder, otherwise assume null/empty.

(reworked pr)
@wing328 wing328 added this to the v2.2.2 milestone Jan 27, 2017
@wing328 wing328 merged commit 8693741 into swagger-api:master Jan 27, 2017
@wing328
Copy link
Copy Markdown
Contributor

wing328 commented Jan 27, 2017

@eblis thanks for the PR, which has been merged into master.

davidgri pushed a commit to davidgri/swagger-codegen that referenced this pull request May 11, 2017
…t the source folder, otherwise assume null/empty. (swagger-api#4626)

(reworked pr)
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