Skip to content

Manual monthly pre-commit update #309

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.

Sign up for GitHub

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
avalentino merged 1 commit into from
Aug 4, 2026
Merged

Manual monthly pre-commit update #309

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .pre-commit-config.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ repos:
- id: end-of-file-fixer
- id: trailing-whitespace
- repo: https://github.com/astral-sh/ruff-pre-commit
rev: v0.15.20
rev: v0.16.1
hooks:
- id: ruff-check
args: ["--fix", "--show-fixes"]
2 changes: 1 addition & 1 deletion docs/conf.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
from datetime import datetime

try:
from sphinx_astropy.conf.v1 import * # noqa: F403
from sphinx_astropy.conf.v1 import *

@eerovaher eerovaher Aug 3, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

F403 is no longer enforced by default.

from sphinx_astropy.conf.v1 import exclude_patterns
except ImportError:
print('ERROR: the documentation requires the sphinx-astropy package to be installed')
Expand Down
6 changes: 5 additions & 1 deletion pyproject.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -91,10 +91,14 @@ extend-select = [
"SIM",
"UP",
]
ignore = [
"BLE001", # temporary
"DTZ", # temporary
"S110", # temporary
]
[tool.ruff.lint.extend-per-file-ignores]
"__init__.py" = [
"F401", # unused-import
"F403", # undefined-local-with-import-star
]
"test_*.py" = ["S101"]

Expand Down
2 changes: 1 addition & 1 deletion setup.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ def guess_next_dev(version: ScmVersion) -> str:
sources = [Path("erfa", "ufunc.c")]
include_dirs = [np.get_include()]
libraries = []
if int(os.environ.get("PYERFA_USE_SYSTEM_LIBERFA", 0)):
if int(os.environ.get("PYERFA_USE_SYSTEM_LIBERFA", "0")):

@eerovaher eerovaher Aug 3, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

PLW1508 (invalid-envvar-default) is now turned on by default, but it was simpler to enforce it than to configure Ruff to ignore it.

print("Using system liberfa")
libraries.append("erfa")
else:
Expand Down
Loading