-
Notifications
You must be signed in to change notification settings - Fork 17
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
chore: Migrate typing to matplotlib v3.8.0 #563
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #563 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 50 50
Lines 2200 2208 +8
Branches 543 545 +2
=========================================
+ Hits 2200 2208 +8
☔ View full report in Codecov by Sentry. |
would resolve #562 |
bc1af47
to
90bc557
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you so much for taking care of this.
Regarding importing of typehints I have two minor change requests though.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alright, let's merge then
there are still a couple of TODOs, see pydata/xarray#8030, but we can upgrade to
matplotlib==3.8.0
.I'll leave it up to my fellow code owners @dkrako and @prassepaul to decide which way to merge. either this one or #561. this PR would resolve #562