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 #4895
Conversation
I'll push a manual fix for the new linting rule |
c5ba7c3
to
869c228
Compare
e9b846f
to
3f7bc52
Compare
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.4.0 → v0.4.3](astral-sh/ruff-pre-commit@v0.4.0...v0.4.3) - [github.com/MarcoGorelli/cython-lint: v0.16.0 → v0.16.2](MarcoGorelli/cython-lint@v0.16.0...v0.16.2)
3f7bc52
to
6902032
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.
This LGTM.
str(type(obj)).split("'")[1], len(obj) | ||
) | ||
f"Expected an array of size (3,), " | ||
f"received {_full_type_name(obj)!r} of length {len(obj)}" |
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.
Why _full_type_name(obj)!r
rather than just _full_type_name(obj)
? Is it just to get the ''
for free?
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.
s it just to get the '' for free?
yes
@@ -290,7 +290,7 @@ def transfer_function(self, value): | |||
if not isinstance(value, valid_types): | |||
raise RuntimeError( | |||
"transfer_function not a valid type, " | |||
"received object of type %s" % type(value) | |||
f"received object of type {type(value)}" |
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.
f"received object of type {type(value)}" | |
f"received object of type {type(value)}." |
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'm trying not to change the result at all, so I'd rather skip this
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.
Fair enough, I'm happy with the PR as is.
updates: