Skip to content
This repository has been archived by the owner on Jan 19, 2024. It is now read-only.

Python Requirements Update #331

Merged
merged 2 commits into from
Aug 24, 2021
Merged

Python Requirements Update #331

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
59f1dea
chore: Updating Python Requirements
edx-requirements-bot Aug 24, 2021
396f0bb
fix: fix the new pylint warnings
UsamaSadiq Aug 24, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion bok_choy/a11y/axe_core_ruleset.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -195,7 +195,7 @@ def customize_ruleset(self, custom_ruleset_file=None):
if not custom_file:
return

with open(custom_file) as additional_rules:
with open(custom_file, encoding="utf8") as additional_rules:
Copy link
Contributor

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.

custom_rules = additional_rules.read()

if "var customRules" not in custom_rules:
Expand Down
2 changes: 1 addition & 1 deletion bok_choy/browser.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -180,7 +180,7 @@ def save_driver_logs(driver, prefix):
file_name = os.path.join(
log_dir, f'{prefix}_{log_type}.log'
)
with open(file_name, 'w') as output_file:
with open(file_name, 'w', encoding="utf8") as output_file:
for line in log:
output_file.write("{}{}".format(dumps(line), '\n'))
except: # pylint: disable=bare-except
Expand Down
2 changes: 1 addition & 1 deletion bok_choy/page_object.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -450,7 +450,7 @@ def _is_document_ready():
).fulfill()
except BrokenPromise:
LOGGER.warning(
u'document.readyState does not become complete for following url: {}'.format(self.url),
'document.readyState does not become complete for following url: {}'.format(self.url),
exc_info=True
)
# If document.readyState does not become complete after a specific time relax the
Expand Down
10 changes: 6 additions & 4 deletions requirements/dev.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
#
# make upgrade
#
astroid==2.6.6
astroid==2.7.2
# via
# -r requirements/needle.txt
# pylint
Expand Down Expand Up @@ -129,7 +129,9 @@ pip-tools==6.2.0
# via -r requirements/pip-tools.txt
platformdirs==2.2.0
# via
# -r requirements/needle.txt
# -r requirements/travis.txt
# pylint
# virtualenv
pluggy==0.13.1
# via
Expand All @@ -146,7 +148,7 @@ py==1.10.0
# tox
pycodestyle==2.7.0
# via -r requirements/needle.txt
pylint==2.9.6
pylint==2.10.2
# via
# -r requirements/needle.txt
# edx-lint
Expand Down Expand Up @@ -216,7 +218,7 @@ sqlparse==0.4.1
# via
# -r requirements/needle.txt
# django
stevedore==3.3.0
stevedore==3.4.0
# via
# -r requirements/needle.txt
# code-annotations
Expand All @@ -236,7 +238,7 @@ tomli==1.2.1
# via
# -r requirements/pip-tools.txt
# pep517
tox==3.24.1
tox==3.24.3
# via
# -r requirements/travis.txt
# tox-battery
Expand Down
10 changes: 7 additions & 3 deletions requirements/needle.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
#
# make upgrade
#
astroid==2.6.6
astroid==2.7.2
# via
# -r requirements/test.txt
# pylint
Expand Down Expand Up @@ -84,6 +84,10 @@ pbr==5.6.0
# stevedore
pillow==8.3.1
# via needle
platformdirs==2.2.0
# via
# -r requirements/test.txt
# pylint
pluggy==0.13.1
# via
# -r requirements/test.txt
Expand All @@ -95,7 +99,7 @@ py==1.10.0
# pytest-forked
pycodestyle==2.7.0
# via -r requirements/test.txt
pylint==2.9.6
pylint==2.10.2
# via
# -r requirements/test.txt
# edx-lint
Expand Down Expand Up @@ -157,7 +161,7 @@ sqlparse==0.4.1
# via
# -r requirements/test.txt
# django
stevedore==3.3.0
stevedore==3.4.0
# via
# -r requirements/test.txt
# code-annotations
Expand Down
8 changes: 5 additions & 3 deletions requirements/test.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
#
# make upgrade
#
astroid==2.6.6
astroid==2.7.2
# via
# pylint
# pylint-celery
Expand Down Expand Up @@ -51,6 +51,8 @@ packaging==21.0
# pytest
pbr==5.6.0
# via stevedore
platformdirs==2.2.0
# via pylint
pluggy==0.13.1
# via pytest
py==1.10.0
Expand All @@ -59,7 +61,7 @@ py==1.10.0
# pytest-forked
pycodestyle==2.7.0
# via -r requirements/test.in
pylint==2.9.6
pylint==2.10.2
# via
# edx-lint
# pylint-celery
Expand Down Expand Up @@ -98,7 +100,7 @@ six==1.16.0
# via edx-lint
sqlparse==0.4.1
# via django
stevedore==3.3.0
stevedore==3.4.0
# via code-annotations
text-unidecode==1.3
# via python-slugify
Expand Down
2 changes: 1 addition & 1 deletion requirements/travis.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ six==1.16.0
# virtualenv
toml==0.10.2
# via tox
tox==3.24.1
tox==3.24.3
# via
# -r requirements/travis.in
# tox-battery
Expand Down