Skip to content

Update Black dependency to fix CI lint#900

Merged
sobolevn merged 2 commits intotypeddjango:masterfrom
intgr:Update-Black-to-fix-CI-lint
Mar 31, 2022
Merged

Update Black dependency to fix CI lint#900
sobolevn merged 2 commits intotypeddjango:masterfrom
intgr:Update-Black-to-fix-CI-lint

Conversation

@intgr
Copy link
Copy Markdown
Collaborator

@intgr intgr commented Mar 30, 2022

Closes #896.

Black 22.3.0 fixes incompatibility with Click 8.1.0: psf/black#2966

This currently causes the CI to fail, e.g. https://github.com/typeddjango/django-stubs/runs/5756075543

Closes typeddjango#896.

Black 22.3.0 fixes incompatibility with Click 8.1.0:
psf/black#2966

This currently causes the CI to fail, e.g. https://github.com/typeddjango/django-stubs/runs/5756075543
args: ["--profile", "black"]
- repo: https://github.com/psf/black
rev: 21.12b0
rev: 22.3.0
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Looks like .pre-commit-config.yaml hasn't been kept in sync with requirements.txt as it should be.

@intgr
Copy link
Copy Markdown
Collaborator Author

intgr commented Mar 30, 2022

I'm not sure why it's still failing, but it's a different message this time.

@sobolevn
Copy link
Copy Markdown
Member

You need to reformat some files with new black version.

@intgr
Copy link
Copy Markdown
Collaborator Author

intgr commented Mar 30, 2022

Ahh! Makes sense, I will do it tomorrow.

@intgr
Copy link
Copy Markdown
Collaborator Author

intgr commented Mar 31, 2022

Fixed, should be ready to merge now.

@sobolevn sobolevn merged commit 212ef40 into typeddjango:master Mar 31, 2022
@sobolevn
Copy link
Copy Markdown
Member

Thank you!

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants