Skip to content
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 #460

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

pre-commit-ci[bot]
Copy link
Contributor

@pre-commit-ci pre-commit-ci bot commented Mar 3, 2025

updates:
- [github.com/astral-sh/ruff-pre-commit: v0.9.4 → v0.9.9](astral-sh/ruff-pre-commit@v0.9.4...v0.9.9)
- [github.com/pre-commit/mirrors-mypy: v1.14.1 → v1.15.0](pre-commit/mirrors-mypy@v1.14.1...v1.15.0)
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from aa5e2fc to 6daf59d Compare March 3, 2025 16:29
@adamltyson
Copy link
Member

The newest version of mypy is raising new errors:

 tests/test_unit/test_validators/test_files_validators.py:97: SyntaxWarning: invalid escape sequence '\d'
  "The provided frame regexp ((0\d*)\.\w+$) did not return "
movement/utils/broadcasting.py:273: error: ParamSpec "KeywordArgs" is unbound  [valid-type]
movement/utils/broadcasting.py:275: error: ParamSpec "KeywordArgs" is unbound  [valid-type]
movement/utils/broadcasting.py:291: error: ParamSpec "KeywordArgs" is unbound  [valid-type]
movement/utils/broadcasting.py:292: error: ParamSpec "KeywordArgs" is unbound  [valid-type]
movement/utils/broadcasting.py:305: error: ParamSpec "KeywordArgs" is unbound  [valid-type]
movement/utils/broadcasting.py:307: error: ParamSpec "KeywordArgs" is unbound  [valid-type]
movement/utils/broadcasting.py:322: error: ParamSpec "KeywordArgs" is unbound  [valid-type]
movement/utils/broadcasting.py:323: error: ParamSpec "KeywordArgs" is unbound  [valid-type]
Found 8 errors in 1 file (checked 83 source files)

I will leave it to @neuroinformatics-unit/behaviour to decide what (if anything) to do about it.

@lochhh
Copy link
Collaborator

lochhh commented Mar 4, 2025

The syntax error is resolved in #452

@niksirbi
Copy link
Member

niksirbi commented Mar 4, 2025

The syntax error is resolved in #452

That fix is currently in main, so this PR can be rebased.

Copy link

sonarqubecloud bot commented Mar 4, 2025

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