Skip to content

ci: Run mypy against typed cycler #26943

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 11, 2023
Merged

Conversation

QuLogic
Copy link
Member

@QuLogic QuLogic commented Sep 28, 2023

PR summary

This tests against the new cycler release candidate, and then drops the flag/comment noted in the ci config.

PR checklist

@QuLogic QuLogic added the CI: testing CI configuration and testing label Sep 28, 2023
@QuLogic QuLogic marked this pull request as ready for review September 29, 2023 00:34
@QuLogic
Copy link
Member Author

QuLogic commented Sep 29, 2023

Un-drafted now that typed cycler is out.

@QuLogic QuLogic added this to the v3.8.1 milestone Sep 29, 2023
@ksunden
Copy link
Member

ksunden commented Oct 2, 2023

We have the ignore_missing_imports=true in pyproject.toml, and removing that has revealed that cycler installed from pypi doesn't include the py.typed sentinel file...

Which is somewhat odd because matplotlib/cycler#88 specifically mentions that file (in the sdist)

I can confirm that building locally the file is not included...

@ksunden
Copy link
Member

ksunden commented Oct 2, 2023

I've opened matplotlib/cycler#98 in response

@QuLogic
Copy link
Member Author

QuLogic commented Oct 7, 2023

OK, I've made a new cycler release with those changes included, and updated pyproject.toml here as well.

@ksunden ksunden merged commit 6d5f286 into matplotlib:main Oct 11, 2023
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Oct 11, 2023
@QuLogic QuLogic deleted the typed-cycler branch October 11, 2023 20:10
story645 added a commit that referenced this pull request Oct 12, 2023
…943-on-v3.8.x

Backport PR #26943 on branch v3.8.x (ci: Run mypy against typed cycler)
@ksunden ksunden mentioned this pull request Nov 2, 2023
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI: testing CI configuration and testing topic: typing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants