Skip to content

Conversation

pre-commit-ci bot and others added 2 commits October 27, 2025 20:44
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.14.1 → v0.14.2](astral-sh/ruff-pre-commit@v0.14.1...v0.14.2)
- [github.com/woodruffw/zizmor-pre-commit: v1.15.2 → v1.16.0](zizmorcore/zizmor-pre-commit@v1.15.2...v1.16.0)
- [github.com/RobertCraigie/pyright-python: v1.1.406 → v1.1.407](RobertCraigie/pyright-python@v1.1.406...v1.1.407)
from fractions import Fraction
from math import inf
from math import nan
from math import sqrt
Copy link
Member

Choose a reason for hiding this comment

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

math is already import for sqrt so there's a slight advantage in using math.inf / math.nan

@Pierre-Sassoulas
Copy link
Member

Thank you for the review @bluetech. It's very strange, I can't merge atm.

image

@bluetech bluetech merged commit 6f9c628 into main Oct 28, 2025
33 checks passed
@bluetech bluetech deleted the pre-commit-ci-update-config branch October 28, 2025 10:30
@bluetech
Copy link
Member

Hmm looks like someone in GitHub doesn't like you :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants