-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[pre-commit.ci] pre-commit autoupdate #16396
Conversation
updates: - [github.com/pre-commit/pre-commit-hooks: v4.5.0 → v4.6.0](pre-commit/pre-commit-hooks@v4.5.0...v4.6.0) - [github.com/astral-sh/ruff-pre-commit: v0.3.4 → v0.4.3](astral-sh/ruff-pre-commit@v0.3.4...v0.4.3) - [github.com/scientific-python/cookie: 2024.01.24 → 2024.04.23](scientific-python/cookie@2024.01.24...2024.04.23)
Thank you for your contribution to Astropy! 🌌 This checklist is meant to remind the package maintainers who will review this pull request of some common things to look for.
|
for more information, see https://pre-commit.ci
Let's update the pre-commit config so that these PRs include the recommended steps to fix a failing PR. Currently our practice is to close this PR, cherry-pick the commits into a new PR where we fix whatever's wrong. This is because the Pre-commit PRs are branches on Astropy main, not from a fork. Let's record this institutional knowledge into the PR message itself. |
@@ -138,7 +138,7 @@ | |||
# Safe in < 2.0, because it deferred to the method. Overridden in >= 2.0. | |||
MASKED_SAFE_FUNCTIONS |= {np.ptp} | |||
# Removed in numpy 2.0. Just an alias to vstack. | |||
MASKED_SAFE_FUNCTIONS |= {np.row_stack} | |||
MASKED_SAFE_FUNCTIONS |= {np.vstack} |
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.
I don't think this change should happen, right?
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.
No, this is ruff trying to fix our code for v2, but we don't want that in this specific case.
Yeah... let's close and PR from fork.
|
updates: