Skip to content

DOC: Added missing dtype attribute to iinfo and finfo docstring#22334

Merged
rgommers merged 3 commits intonumpy:mainfrom
MatteoRaso:doc_fix
Sep 27, 2022
Merged

DOC: Added missing dtype attribute to iinfo and finfo docstring#22334
rgommers merged 3 commits intonumpy:mainfrom
MatteoRaso:doc_fix

Conversation

@MatteoRaso
Copy link
Copy Markdown
Contributor

@MatteoRaso MatteoRaso commented Sep 25, 2022

finfo() and iinfo() both have a dtype attribute, which was undocumented before this commit.

Closes gh-22330

skip ci

finfo() and iinfo() both have a dtype attribute,
which was undocumented before this commit.

skip ci
Copy link
Copy Markdown
Member

@rgommers rgommers left a comment

Choose a reason for hiding this comment

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

Thanks @MatteoRaso. I added a couple of suggestions to make the docstring more precise.

@rgommers rgommers changed the title DOC: Added missing attribute (#22330) DOC: Added missing dtype attribute to iinfo and finfo docstring Sep 25, 2022
Copy link
Copy Markdown
Member

@rgommers rgommers left a comment

Choose a reason for hiding this comment

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

LGTM now and doc build is green, in it goes. Thanks @MatteoRaso

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.

DOC: Document finfo() result holds a dtype attribute

2 participants