-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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
[Style] update flake8 to latest and update requirements #48023
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Cheyu Wu <[email protected]>
…an unexpected keyword argument 'unsafe_hash'. Signed-off-by: Cheyu Wu <[email protected]>
flake8-comprehensions==3.10.1, | ||
flake8-quotes==2.0.0, | ||
flake8-comprehensions==3.15.0, | ||
flake8-quotes==3.4.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.
you need to upgrade bugbear too, and update the python/requirements_copmiled.txt
file
here is an example of the dependency update of flake (without the additional ignore list though).
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.
OK, no problem. Thanks for your advice
Signed-off-by: Cheyu Wu <[email protected]>
….txt Signed-off-by: Cheyu Wu <[email protected]>
python/requirements_compiled.txt
Outdated
@@ -521,17 +521,21 @@ filelock==3.13.1 | |||
# torch | |||
# transformers | |||
# virtualenv | |||
flake8==3.9.1 | |||
flake8==7.1.1; python_version > '3.8' | |||
flake8==3.9.1; python_version <= '3.8' |
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.
we do not support <=3.8
anymore. can you just remove those?
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.
Sure, I will handle that
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.
Well, The pipeline will be checked for Python 3.8.
I'm not sure how to resolve this issue since the CI pipeline will pull Python 3.8.
Signed-off-by: Cheyu Wu <[email protected]>
Why are these changes needed?
Related issue number
Closes #48022
Checks
git commit -s
) in this PR.scripts/format.sh
to lint the changes in this PR.method in Tune, I've added it in
doc/source/tune/api/
under thecorresponding
.rst
file.cc @MortalHappiness