-
Notifications
You must be signed in to change notification settings - Fork 3.8k
Pin black version #8139
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
Pin black version #8139
Conversation
|
cc @areusch @tkonolige re: your work in updating the Docker images. The chosen pinned version comes from the version being currently used by Recently released version of |
| RUN apt-get update && apt-get install -y doxygen graphviz | ||
|
|
||
| RUN pip3 install cpplint pylint==2.4.4 mypy black | ||
| RUN pip3 install cpplint pylint==2.4.4 mypy black==20.8b1 |
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.
could you also add this in python/gen_requirements.py (the CONSTRAINTS global). i've been focused on CI work but will pick that up again soon.
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.
Done. Thank you for the review.
ed298d2 to
d6ed645
Compare
This commit pins the black version to provide stability. It is expected that the pinned version will be moved forward periodically. Change-Id: Ied866bff85a1a832959bc1d4673a7fdec68128a7
d6ed645 to
30a149a
Compare
leandron
left a comment
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.
LGTM
areusch
left a comment
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 @NicolaLancellotti !
This commit pins the black version to provide stability. It is expected that the pinned version will be moved forward periodically. Change-Id: Ied866bff85a1a832959bc1d4673a7fdec68128a7
This commit pins the black version to provide stability. It is expected that the pinned version will be moved forward periodically. Change-Id: Ied866bff85a1a832959bc1d4673a7fdec68128a7
This pr pins the black version to provide stability.
It is expected that the pinned version will be moved forward periodically.