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(bindings/python): Update type annotations #4630

Merged
merged 1 commit into from
May 19, 2024
Merged

Conversation

3ok
Copy link
Contributor

@3ok 3ok commented May 19, 2024

This PR does not close any issues per se but mostly makes the Python binding type stubs up-to-date with changes such as #4384 and #4360

The stubs now pass both mypy --strict and stubtest checks

Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thank you! We are looking forward to pyo3's auto stub generation support.

@Xuanwo
Copy link
Member

Xuanwo commented May 19, 2024

The stubs now pass both mypy --strict and stubtest checks

Are you willing to start a new PR to add those checks in our python CI?

@Xuanwo Xuanwo merged commit 87c558a into apache:main May 19, 2024
54 checks passed
@3ok
Copy link
Contributor Author

3ok commented May 19, 2024

The stubs now pass both mypy --strict and stubtest checks

Are you willing to start a new PR to add those checks in our python CI?

Sure thing, I'll try to take a look when I have some time

@3ok 3ok deleted the improve-stubs branch May 19, 2024 14:56
George-Miao pushed a commit to George-Miao/opendal that referenced this pull request Jun 5, 2024
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