-
-
Notifications
You must be signed in to change notification settings - Fork 78
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
[deps] Upgraded QA dependencies #342
Conversation
pandafy
commented
Jun 13, 2023
- Bummped black~=23.3.0
- Bumped flake8~=6.0.0
- Bumped readme-renderer~=37.3.0
- Bumped coveralls~=3.3.0
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.
Please rebase on the latest master.
Can you check whether upgrading black changes the formatting in other modules (eg: controller)?
4ad3791
to
7af2da1
Compare
Yes, it does. |
Can you please look if there is there any flag we can add to avoid that? Maybe when they changed the default they provided a way to override it. |
@nemesifier I cannot find such a flag in https://black.readthedocs.io/en/stable/usage_and_configuration/the_basics.html#command-line-options |
Did you mean using the Example from the [black documentation]:(https://black.readthedocs.io/en/stable/usage_and_configuration/the_basics.html#versions)
|
- Bummped black~=23.3.0 - Bumped flake8~=6.0.0 - Bumped readme-renderer~=37.3.0 - Bumped coveralls~=3.3.0
7af2da1
to
3edcd67
Compare
Handled this in #389. |