Skip to content

[pre-commit.ci] pre-commit autoupdate#354

Merged
gaborbernat merged 3 commits intomainfrom
pre-commit-ci-update-config
Apr 16, 2024
Merged

[pre-commit.ci] pre-commit autoupdate#354
gaborbernat merged 3 commits intomainfrom
pre-commit-ci-update-config

Conversation

@pre-commit-ci
Copy link
Copy Markdown
Contributor

@pre-commit-ci pre-commit-ci bot commented Apr 15, 2024

@kemzeb
Copy link
Copy Markdown
Collaborator

kemzeb commented Apr 15, 2024

Looks like the CI failures are due to Ruff now forcing us to string append when we need to prepend in some areas.

@kemzeb
Copy link
Copy Markdown
Collaborator

kemzeb commented Apr 15, 2024

Looking at this further, the problem should be with PLR6104. It is only accessible in preview mode, but since we enable it here this should be the problem.

@gaborbernat
Copy link
Copy Markdown
Member

Looking at this further, the problem should be with PLR6104. It is only accessible in preview mode, but since we enable it here this should be the problem.

WE can add noqa for these exampels 👍

@gaborbernat gaborbernat merged commit 1bebb55 into main Apr 16, 2024
@gaborbernat gaborbernat deleted the pre-commit-ci-update-config branch April 16, 2024 23:08
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.

2 participants