Skip to content

Fixed array_t type hint - #5353

Closed
gentlegiantJGC wants to merge 3 commits into
pybind:masterfrom
gentlegiantJGC:fix-array_t-stub
Closed

gentlegiantJGC wants to merge 3 commits into
pybind:masterfrom
gentlegiantJGC:fix-array_t-stub

Conversation

@gentlegiantJGC

Copy link
Copy Markdown
Contributor

Description

Fixed type hint for pybind11::array_t
Fixes #5352

Suggested changelog entry:

Fixed type hint for `pybind11::array_t`

@Skylion007 Skylion007 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I'm kind of shocked none of the linters complained before.

@Skylion007 Skylion007 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Actually, this is the same as: https://github.com/pybind/pybind11/pull/5212/files We should probably merge that PR.

@gentlegiantJGC

Copy link
Copy Markdown
Contributor Author

I missed that pull request.
Their version looks better.

@gentlegiantJGC
gentlegiantJGC deleted the fix-array_t-stub branch September 3, 2024 15:39
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.

[BUG]: array_t stub is incorrect.

2 participants