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

feat: use openapi-generator to generate client #58

Merged
merged 6 commits into from
Mar 23, 2019

Conversation

tomplus
Copy link
Owner

@tomplus tomplus commented Jan 11, 2019

These changes were generated by openapi-generator instead of swagger-codegen.

Ref: kubernetes-client/gen#93

@@ -1,2 +0,0 @@
Requested Commit: f9b2839a3076f26db1b8fc61655a26662f2552ee
Copy link
Owner Author

Choose a reason for hiding this comment

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

I had to delete kubernetes_asyncio/.swagger-codegen and rename kubernetes_asyncio/.swagger-codegen-ignore to kubernetes_asyncio/.openapi-generator-ignore.

@codecov-io
Copy link

codecov-io commented Jan 19, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@d57e9e9). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #58   +/-   ##
=========================================
  Coverage          ?   94.39%           
=========================================
  Files             ?       20           
  Lines             ?     1355           
  Branches          ?        0           
=========================================
  Hits              ?     1279           
  Misses            ?       76           
  Partials          ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d57e9e9...c0c04ee. Read the comment docs.

@@ -61,6 +61,7 @@ def __aiter__(self):
with patch('kubernetes_asyncio.client.api_client.rest', mock):

api_client = WsApiClient()
api_client.configuration.host = 'https://localhost'
Copy link
Owner Author

Choose a reason for hiding this comment

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

because the openapi-generator uses http://localhost as a default hostname and the swagger-codegen used https://localhost.

Copy link
Owner Author

Choose a reason for hiding this comment

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

@tomplus tomplus merged commit 91ade8a into master Mar 23, 2019
@tomplus tomplus changed the title [WIP] feat: use openapi-generator to generate client feat: use openapi-generator to generate client Mar 23, 2019
@tomplus tomplus deleted the feat/openapi-generator branch February 26, 2020 22:46
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.

2 participants