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
Python Requirements Update #206
Python Requirements Update #206
Conversation
Codecov Report
@@ Coverage Diff @@
## master #206 +/- ##
=======================================
Coverage 77.29% 77.29%
=======================================
Files 21 21
Lines 1026 1026
=======================================
Hits 793 793
Misses 233 233
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
@@ -14,7 +14,7 @@ def get_version(*file_paths): | |||
Extract the version string from the file at the given relative path fragments. | |||
""" | |||
filename = os.path.join(os.path.dirname(__file__), *file_paths) | |||
version_file = open(filename).read() # pylint: disable=consider-using-with | |||
version_file = open(filename, encoding="utf8").read() # pylint: disable=consider-using-with |
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.
pylint version 2.10.0 deprecated the use of open() without explicitly specifying any encoding. See pylint-dev/pylint#3826 for details.
@timmc-edx I'll create another PR to add six in the requirements for now to resolve the failing Jenkins job issue. |
Thanks! |
Python requirements update.Please review the changelogs for the upgraded packages.