-
Notifications
You must be signed in to change notification settings - Fork 703
chore: update pyi files #6861
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
chore: update pyi files #6861
Conversation
Signed-off-by: yihong0618 <zouzou0208@gmail.com>
Signed-off-by: yihong0618 <zouzou0208@gmail.com>
|
since its not a opendal issue its from uv and update uv python no issue here, so this patch will only update the python desc |
Signed-off-by: yihong0618 <zouzou0208@gmail.com>
Xuanwo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should find a way to make sure those pyi files are updated after the core changes. Would you like to add a CI for Python, similar to what we have for Node.js, that checks the diff after generation?
opendal/.github/workflows/ci_bindings_nodejs.yml
Lines 85 to 86 in 6ee74a5
| - name: Check diff | |
| run: git diff --exit-code |
of course will add it in this patch |
Signed-off-by: yihong0618 <zouzou0208@gmail.com>
Signed-off-by: yihong0618 <zouzou0208@gmail.com>
Signed-off-by: yihong0618 <zouzou0208@gmail.com>
Signed-off-by: yihong0618 <zouzou0208@gmail.com>
Signed-off-by: yihong0618 <zouzou0208@gmail.com>
|
close as play pyo3-stub-gen on ci and local are all different will bring it back when I have time or others can take it. |
Which issue does this PR close?
Closes #6860
Rationale for this change
What changes are included in this PR?
Are there any user-facing changes?
also update pyi with just