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

Skip to content

MNT: Update all pre-commit hooks#31522

Merged
QuLogic merged 1 commit intomatplotlib:mainfrom
QuLogic:prek-update
Apr 23, 2026
Merged

MNT: Update all pre-commit hooks#31522
QuLogic merged 1 commit intomatplotlib:mainfrom
QuLogic:prek-update

Conversation

@QuLogic
Copy link
Copy Markdown
Member

@QuLogic QuLogic commented Apr 18, 2026

PR summary

When we moved off the app to running in Actions, we lost the auto update and things fell behind. Now set up Dependabot to automate these like everything else.

Also bump the minimum mypy, as such old versions no longer work.

AI Disclosure

None

PR checklist

@QuLogic QuLogic added this to the v3.11.0 milestone Apr 18, 2026
timhoffm added a commit to timhoffm/matplotlib that referenced this pull request Apr 18, 2026
Closes matplotlib#31522.

The root cause is that Axes3D inherits a lot of functionality from Axes
that does not work for 3D. This is a design flaw we cannot easily fix.

With this PR we have at least a reasonable mechanism in place to flag
not-supported methods. When we get aware of additional methods, we can
easily add them.
timhoffm added a commit to timhoffm/matplotlib that referenced this pull request Apr 18, 2026
Closes matplotlib#31522.

The root cause is that Axes3D inherits a lot of functionality from Axes
that does not work for 3D. This is a design flaw we cannot easily fix.

With this PR we have at least a reasonable mechanism in place to flag
not-supported methods. When we get aware of additional methods, we can
easily add them.
timhoffm added a commit to timhoffm/matplotlib that referenced this pull request Apr 19, 2026
Closes matplotlib#31522.

The root cause is that Axes3D inherits a lot of functionality from Axes
that does not work for 3D. This is a design flaw we cannot easily fix.

With this PR we have at least a reasonable mechanism in place to flag
not-supported methods. When we get aware of additional methods, we can
easily add them.
timhoffm added a commit to timhoffm/matplotlib that referenced this pull request Apr 19, 2026
Closes matplotlib#31522.

The root cause is that Axes3D inherits a lot of functionality from Axes
that does not work for 3D. This is a design flaw we cannot easily fix.

With this PR we have at least a reasonable mechanism in place to flag
not-supported methods. When we get aware of additional methods, we can
easily add them.
timhoffm added a commit to timhoffm/matplotlib that referenced this pull request Apr 19, 2026
Closes matplotlib#31522.

The root cause is that Axes3D inherits a lot of functionality from Axes
that does not work for 3D. This is a design flaw we cannot easily fix.

With this PR we have at least a reasonable mechanism in place to flag
not-supported methods. When we get aware of additional methods, we can
easily add them.
@QuLogic QuLogic closed this in f8a68b7 Apr 21, 2026
@tacaswell tacaswell reopened this Apr 21, 2026
@tacaswell
Copy link
Copy Markdown
Member

This got closed by a typo in a commit message.

Comment thread .github/dependabot.yml Outdated
- package-ecosystem: "pre-commit"
directory: "/"
schedule:
interval: "weekly"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
interval: "weekly"
interval: "monthly"

I'd say these don't need updating weekly, so better to go with monthly and reduce the number of auto update PRs that end up being opened.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Sure.

When we moved off the app to running in Actions, we lost the auto update
and things fell behind. Now set up Dependabot to automate these like
everything else.

Also bump the minimum mypy, as such old versions no longer work.
@QuLogic QuLogic merged commit c0c18c9 into matplotlib:main Apr 23, 2026
40 of 41 checks passed
@QuLogic QuLogic deleted the prek-update branch April 23, 2026 00:14
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