Skip to content

GH-49452: [Python] Reintroduce docstring injection for stubfiles#49453

Open
rok wants to merge 3 commits intoapache:mainfrom
rok:docstring-injection
Open

GH-49452: [Python] Reintroduce docstring injection for stubfiles#49453
rok wants to merge 3 commits intoapache:mainfrom
rok:docstring-injection

Conversation

@rok
Copy link
Member

@rok rok commented Mar 4, 2026

Rationale for this change

Warning: should not be merged before #49259.
See #49452 and #48618

What changes are included in this PR?

Adds a wheel build time script to populate stubfiles with runtime docstrings.

Are these changes tested?

Not yet.

Are there any user-facing changes?

Users will get docstrings.

@github-actions
Copy link

github-actions bot commented Mar 4, 2026

⚠️ GitHub issue #49452 has been automatically assigned in GitHub to PR creator.

@rok rok force-pushed the docstring-injection branch from 095ee4c to 2056fec Compare March 9, 2026 17:37
@rok
Copy link
Member Author

rok commented Mar 9, 2026

@raulcd this is ready for review

@rok rok requested review from assignUser, jonkeane and kou as code owners March 10, 2026 00:34
@rok rok force-pushed the docstring-injection branch from 2f0f841 to f885111 Compare March 10, 2026 00:57
@raulcd
Copy link
Member

raulcd commented Mar 10, 2026

@github-actions crossbow submit wheel*-cp313-*

@github-actions
Copy link

Revision: f885111

Submitted crossbow builds: ursacomputing/crossbow @ actions-c1678cd8dd

Task Status
wheel-macos-monterey-cp313-cp313-amd64 GitHub Actions
wheel-macos-monterey-cp313-cp313-arm64 GitHub Actions
wheel-macos-monterey-cp313-cp313t-amd64 GitHub Actions
wheel-macos-monterey-cp313-cp313t-arm64 GitHub Actions
wheel-manylinux-2-28-cp313-cp313-amd64 GitHub Actions
wheel-manylinux-2-28-cp313-cp313-arm64 GitHub Actions
wheel-manylinux-2-28-cp313-cp313t-amd64 GitHub Actions
wheel-manylinux-2-28-cp313-cp313t-arm64 GitHub Actions
wheel-musllinux-1-2-cp313-cp313-amd64 GitHub Actions
wheel-musllinux-1-2-cp313-cp313-arm64 GitHub Actions
wheel-musllinux-1-2-cp313-cp313t-amd64 GitHub Actions
wheel-musllinux-1-2-cp313-cp313t-arm64 GitHub Actions
wheel-windows-cp313-cp313-amd64 GitHub Actions
wheel-windows-cp313-cp313t-amd64 GitHub Actions

Copy link
Member

@raulcd raulcd left a comment

Choose a reason for hiding this comment

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

@rok the new check on the ci/scripts/python_wheel_validate_contents.py doesn't seem to work :(

@github-actions github-actions bot added awaiting changes Awaiting changes awaiting change review Awaiting change review and removed awaiting committer review Awaiting committer review awaiting changes Awaiting changes labels Mar 10, 2026
@rok rok force-pushed the docstring-injection branch from d638b95 to 10509c4 Compare March 10, 2026 12:49
@rok
Copy link
Member Author

rok commented Mar 10, 2026

@github-actions crossbow submit wheel*-cp313-*

@github-actions
Copy link

Revision: 10509c4

Submitted crossbow builds: ursacomputing/crossbow @ actions-f3f679ce77

Task Status
wheel-macos-monterey-cp313-cp313-amd64 GitHub Actions
wheel-macos-monterey-cp313-cp313-arm64 GitHub Actions
wheel-macos-monterey-cp313-cp313t-amd64 GitHub Actions
wheel-macos-monterey-cp313-cp313t-arm64 GitHub Actions
wheel-manylinux-2-28-cp313-cp313-amd64 GitHub Actions
wheel-manylinux-2-28-cp313-cp313-arm64 GitHub Actions
wheel-manylinux-2-28-cp313-cp313t-amd64 GitHub Actions
wheel-manylinux-2-28-cp313-cp313t-arm64 GitHub Actions
wheel-musllinux-1-2-cp313-cp313-amd64 GitHub Actions
wheel-musllinux-1-2-cp313-cp313-arm64 GitHub Actions
wheel-musllinux-1-2-cp313-cp313t-amd64 GitHub Actions
wheel-musllinux-1-2-cp313-cp313t-arm64 GitHub Actions
wheel-windows-cp313-cp313-amd64 GitHub Actions
wheel-windows-cp313-cp313t-amd64 GitHub Actions

@rok rok force-pushed the docstring-injection branch from 10509c4 to e4903eb Compare March 10, 2026 13:18
@rok
Copy link
Member Author

rok commented Mar 10, 2026

@github-actions crossbow submit wheel*-cp313-*

@github-actions
Copy link

Revision: e4903eb

Submitted crossbow builds: ursacomputing/crossbow @ actions-a33b20ecd0

Task Status
wheel-macos-monterey-cp313-cp313-amd64 GitHub Actions
wheel-macos-monterey-cp313-cp313-arm64 GitHub Actions
wheel-macos-monterey-cp313-cp313t-amd64 GitHub Actions
wheel-macos-monterey-cp313-cp313t-arm64 GitHub Actions
wheel-manylinux-2-28-cp313-cp313-amd64 GitHub Actions
wheel-manylinux-2-28-cp313-cp313-arm64 GitHub Actions
wheel-manylinux-2-28-cp313-cp313t-amd64 GitHub Actions
wheel-manylinux-2-28-cp313-cp313t-arm64 GitHub Actions
wheel-musllinux-1-2-cp313-cp313-amd64 GitHub Actions
wheel-musllinux-1-2-cp313-cp313-arm64 GitHub Actions
wheel-musllinux-1-2-cp313-cp313t-amd64 GitHub Actions
wheel-musllinux-1-2-cp313-cp313t-arm64 GitHub Actions
wheel-windows-cp313-cp313-amd64 GitHub Actions
wheel-windows-cp313-cp313t-amd64 GitHub Actions

@rok
Copy link
Member Author

rok commented Mar 10, 2026

@raulcd the docstring presence check was apparently too strict. I think this is ready for review now.

@rok rok linked an issue Mar 10, 2026 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Python] Reintroduce docstring injection for stubfiles

2 participants