Skip to content

Auto-update pre-commit hooks #300

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

Merged
merged 1 commit into from
Nov 30, 2022
Merged

Conversation

github-actions[bot]
Copy link
Contributor

Update versions of tools in pre-commit configs to latest version

@github-actions github-actions bot added the dependencies Pull requests that update a dependency file label Nov 24, 2022
@github-actions github-actions bot force-pushed the update/pre-commit-autoupdate branch from 3165fad to 73217f7 Compare November 30, 2022 02:59
@sobolevn sobolevn merged commit 9672ba7 into master Nov 30, 2022
@@ -24,7 +24,7 @@ repos:
hooks:
- id: black
- repo: https://github.com/PyCQA/flake8
rev: 5.0.4
rev: 6.0.0
Copy link
Contributor

Choose a reason for hiding this comment

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

This seems to break our CI.

    Collecting flake8-pyi==22.10.0
      Downloading flake8_pyi-22.10.0-py37-none-any.whl (24 kB)
    
    The conflict is caused by:
        The user requested flake8 6.0.0 (from /home/runner/.cache/pre-commit/repovn0unijf)
        flake8-pyi 22.10.0 depends on flake8<6.0.0 and >=3.2.1

Do you know why CI doesn't run in this pull request? Running the CI would have detected this issue.

Copy link
Contributor

Choose a reason for hiding this comment

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

Fixed in #303

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Development

Successfully merging this pull request may close these issues.

2 participants