Skip to content

MAINT: Add missing placeholder annotations#18302

Merged
charris merged 4 commits intonumpy:masterfrom
BvB93:missing-attr
Feb 2, 2021
Merged

MAINT: Add missing placeholder annotations#18302
charris merged 4 commits intonumpy:masterfrom
BvB93:missing-attr

Conversation

@BvB93
Copy link
Copy Markdown
Member

@BvB93 BvB93 commented Feb 2, 2021

This PR adds (placeholder) annotations for a few objects that were missed in #17104.

In the wake of #18270 I did a double check of all placeholder annotations and identified a few
that were previously missing, mostly module-level __all__ attributes.

In addition there were two groups of public functions that were missing from their module's __all__.
This has now been rectified:

  • np.ctypeslib.as_ctypes_type
  • Five functions in np.rec.

@BvB93
Copy link
Copy Markdown
Member Author

BvB93 commented Feb 2, 2021

12d9967 and c21509a would qualify for a backport in my opinion.
The remaining two commits aren't nearly as important; I'd say that can remain exclusive to >= 1.21 unless someone has a strong opinion about.

@charris
Copy link
Copy Markdown
Member

charris commented Feb 2, 2021

It would be best if you made the backport PRs.

@BvB93
Copy link
Copy Markdown
Member Author

BvB93 commented Feb 2, 2021

It would be best if you made the backport PRs.

Will do; I'll create the backport after this PR has been merged.

@seberg seberg removed the 09 - Backport-Candidate PRs tagged should be backported label Feb 2, 2021
@charris charris merged commit 80ffa42 into numpy:master Feb 2, 2021
@charris
Copy link
Copy Markdown
Member

charris commented Feb 2, 2021

Thanks Bas.

@BvB93 BvB93 deleted the missing-attr branch February 2, 2021 22:09
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.

3 participants