-
-
Notifications
You must be signed in to change notification settings - Fork 85
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
Fix CI | Update black version in pre-commit #137
Conversation
This fixes pallets/click#2225
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.
thanks @Bobronium, do you mind pushing black
's changes as well so checks can pass?
Python version is updated to satisfy black constraint The current project's Python requirement (>=3.6,<4.0) is not compatible with some of the required packages Python requirement: - black requires Python >=3.6.2, so it will not be satisfied for Python >=3.6,<3.6.2
@branchvincent, done! Had to update python minor version as well. |
I guess CI hasn't been working for a while. poetry installation fails, as well as python 3.6 on macOS.
Turns out, I confused install-poetry.py with get-poetry.py |
Co-authored-by: Branch Vincent <branchevincent@gmail.com>
tests fail since the latest pre-release of poetry dropped 3.6 support, we could pin to I can handle those issues in another PR, getting linting to pass here is more than sufficient |
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.
thanks a lot!
This fixes CI failure caused by pallets/click#2225