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

Set swagger-parser to version 2.0.4 #787

Merged
merged 1 commit into from
Sep 6, 2018

Conversation

jmini
Copy link
Member

@jmini jmini commented Aug 11, 2018

PR checklist

  • Read the contribution guidelines.
  • Ran the shell 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). Windows batch files can be found in .\bin\windows\.
  • Filed the PR against the correct branch: master, 4.0.x. Default: master.
  • Copied the technical committee: @OpenAPITools/generator-core-team

Description of the PR

With #696 swagger-parser was updated to a custom version.
This PR set it to the official 2.0.2 version (see swagger-api/swagger-parser#780)

@jmini
Copy link
Member Author

jmini commented Aug 11, 2018

I think we need to put this PR on hold until swagger-api/swagger-parser#799 is solved.

@wing328
Copy link
Member

wing328 commented Aug 12, 2018

@jmini 👌 thanks for looking into that.

@jmini jmini added the WIP Work in Progress label Aug 14, 2018
@jmini
Copy link
Member Author

jmini commented Aug 14, 2018

This is "ON HOLD", we will move to 2.0.3 when it is ready.

@jmini jmini mentioned this pull request Aug 20, 2018
@jmini jmini changed the title Set swagger-parser to version 2.0.2 Set swagger-parser to version 2.0.3 Sep 1, 2018
@jmini
Copy link
Member Author

jmini commented Sep 1, 2018

When Swagger Parser 2.0.3 is published, this PR can be updated to use the new version.

@jmini jmini force-pushed the change_swagger-parser_version branch from 9f74c51 to bf3ff65 Compare September 5, 2018 20:05
@jmini jmini changed the title Set swagger-parser to version 2.0.3 Set swagger-parser to version 2.0.4 Sep 5, 2018
@jmini
Copy link
Member Author

jmini commented Sep 5, 2018

2.0.4 was released. This PR use it in OpenAPI-Generator

Copy link
Member

@wing328 wing328 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Swagger Parser 2.0.3 release has issues and that's why we skip it: https://github.com/swagger-api/swagger-parser/releases/tag/v2.0.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants