Skip to content

Conversation

@Yang-33
Copy link
Contributor

@Yang-33 Yang-33 commented May 6, 2023

This pull request replaces all clients and models in line-bot-sdk-python with new ones generated by OpenApi generator.

  1. generator is based on PythonNextgenClientCodegen.java in https://github.com/OpenAPITools/openapi-generator. This pull request modifies it to customize discriminator's behavior.
  2. This pull request deletes old documents since it's no longer maintained yet.
  3. Samples and tests are updated. Everyone can look at usage of the latest sdk and test it.

There are some remaining tasks, but it's sufficient to replace because we easily update sdk more quickly than before, thanks to line-openapi and open api generator.

  • Unknown json are not serialized to object because there is no fallback except for webhook event. though line-bot-sdk-java has it...
  • there is no useful documentation in this repository. Some generated document can help developers, but it's not perfect. We should improve it after this change.

resolve #431

@tokuhirom tokuhirom requested review from louis70109 and tokuhirom June 1, 2023 02:06
steps:
- uses: actions/checkout@v3
with:
submodules: true
Copy link
Contributor

Choose a reason for hiding this comment

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

Note: Managing openapi repo as a submodule

strategy:
matrix:
python-version: ['3.7', '3.8', '3.9', '3.10']
python-version: ['3.7', '3.8', '3.9', '3.10', '3.11']
Copy link
Contributor

Choose a reason for hiding this comment

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

Start supporting 3.11

@Yang-33 Yang-33 marked this pull request as ready for review June 7, 2023 23:20
@Yang-33 Yang-33 force-pushed the use-openapi-generator branch from 3e72a68 to ee5eefb Compare June 9, 2023 14:30
@tokuhirom
Copy link
Contributor

@louis70109 could you review this?

@louis70109
Copy link
Member

@tokuhirom ok, let me check~

Copy link
Contributor

@tokuhirom tokuhirom left a comment

Choose a reason for hiding this comment

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

LGTM~

@Yang-33
Copy link
Contributor Author

Yang-33 commented Jun 29, 2023

Thank you for reviewing this change, @tokuhirom @louis70109 !
I'll write short migration guide in the next release note.

@Yang-33 Yang-33 merged commit fe859ad into line:master Jun 29, 2023
@Yang-33 Yang-33 deleted the use-openapi-generator branch June 29, 2023 05:32
Yang-33 added a commit to Yang-33/line-bot-sdk-python that referenced this pull request Jun 30, 2023
Yang-33 added a commit to Yang-33/line-bot-sdk-python that referenced this pull request Jun 30, 2023
Yang-33 added a commit that referenced this pull request Jun 30, 2023
* REVERT-3 Revert "chore(deps): update dependency org.apache.maven.plugins:maven-surefire-plugin to v3 (#451)"

This reverts commit a1203b9.

* REVERT-3 Revert "chore(deps): update dependency org.apache.maven.plugins:maven-jar-plugin to v3 (#450)"

This reverts commit e205a65.

* REVERT-3 Revert "fix(deps): update dependency org.openapitools:openapi-generator to v6.6.0 (#449)"

This reverts commit e3a1cae.

* REVERT-3 Revert "fix(deps): update dependency ch.qos.logback:logback-classic to v1.4.8 (#446)"

This reverts commit bc25f75.

* REVERT-3 Revert "chore(deps): update dependency org.codehaus.mojo:build-helper-maven-plugin to v3.4.0 (#445)"

This reverts commit 822f400.

* REVERT-3 Revert "chore(deps): update dependency org.apache.maven.plugins:maven-surefire-plugin to v2.22.2 (#443)"

This reverts commit 2b8d95e.

* REVERT-3 Revert "chore(deps): update dependency org.apache.maven.plugins:maven-jar-plugin to v2.6 (#442)"

This reverts commit c443a78.

* REVERT-3 Revert "Modify README.rst to pass validation (#441)"

This reverts commit 5e4078e.

* REVERT-3 Revert "chore(deps): update dependency org.apache.maven.plugins:maven-enforcer-plugin to v3.3.0 (#440)"

This reverts commit 1a4f0bf.

* REVERT-3 Revert "chore(deps): update dependency org.apache.maven.plugins:maven-compiler-plugin to v3.11.0 (#439)"

This reverts commit 89e00a7.

* REVERT-3 Revert "Use OpenAPI generator (#432)"

This reverts commit fe859ad.

* REVERT-3 2.4.3
@Yang-33 Yang-33 mentioned this pull request Jul 3, 2023
10 tasks
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.

Start to implement code generation based on LINE OpenAPI specification

3 participants