Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Include data kwarg in pyi stubs#25632

Merged
tacaswell merged 2 commits intomatplotlib:mainfrom
ksunden:data_kwarg
May 1, 2023
Merged

Include data kwarg in pyi stubs#25632
tacaswell merged 2 commits intomatplotlib:mainfrom
ksunden:data_kwarg

Conversation

@ksunden
Copy link
Copy Markdown
Member

@ksunden ksunden commented Apr 5, 2023

PR Summary

Allows for stubtest to check the consistency of many prominent user facing methods

Currently no type hint is provided for data, a follow up issue will be filed detailing what needs to happen to make that have a useful type hint.

PR Checklist

Documentation and Tests

  • Has pytest style unit tests (and pytest passes)
  • Documentation is sphinx and numpydoc compliant (the docs should build without error).
  • New plotting related features are documented with examples.

Release Notes

  • New features are marked with a .. versionadded:: directive in the docstring and documented in doc/users/next_whats_new/
  • API changes are marked with a .. versionchanged:: directive in the docstring and documented in doc/api/next_api_changes/
  • Release notes conform with instructions in next_whats_new/README.rst or next_api_changes/README.rst

Allows for stubtest to check the consistency of many prominent user facing methods

Currently no type hint is provided for data, a follow up issue will be filed detailing what needs to happen to make that have a useful type hint.
@oscargus oscargus added this to the v3.8.0 milestone Apr 20, 2023
@ksunden
Copy link
Copy Markdown
Member Author

ksunden commented May 1, 2023

Gentle nudge on this one, it should be relatively non-controversial I think, just ensuring that stubtest can work on these

@tacaswell tacaswell merged commit e458cb2 into matplotlib:main May 1, 2023
@ksunden ksunden deleted the data_kwarg branch May 1, 2023 21:04
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.

4 participants