Skip to content

Commit

Permalink
chore: remove lxml constraint (#2649)
Browse files Browse the repository at this point in the history
The `edx-i18n-tools` package has been upgraded and we no longer need this constraint.

Relevant `edx-i18n-tools` PR: openedx/i18n-tools#146.
  • Loading branch information
justinhynes authored Dec 4, 2024
1 parent 2e14f53 commit cbdc38b
Show file tree
Hide file tree
Showing 7 changed files with 37 additions and 21 deletions.
13 changes: 9 additions & 4 deletions requirements/all.txt
Original file line number Diff line number Diff line change
Expand Up @@ -33,11 +33,11 @@ bleach==6.2.0
# via
# -r requirements/dev.txt
# -r requirements/production.txt
boto3==1.35.73
boto3==1.35.74
# via
# -r requirements/production.txt
# django-ses
botocore==1.35.73
botocore==1.35.74
# via
# -r requirements/production.txt
# boto3
Expand Down Expand Up @@ -492,13 +492,18 @@ jmespath==1.0.1
# -r requirements/production.txt
# boto3
# botocore
lxml[html-clean]==5.1.1
lxml[html-clean]==5.3.0
# via
# -c requirements/constraints.txt
# -r requirements/dev.txt
# -r requirements/production.txt
# edx-credentials-themes
# edx-i18n-tools
# lxml-html-clean
lxml-html-clean==0.4.1
# via
# -r requirements/dev.txt
# -r requirements/production.txt
# lxml
markdown==3.7
# via
# -r requirements/dev.txt
Expand Down
6 changes: 4 additions & 2 deletions requirements/base.txt
Original file line number Diff line number Diff line change
Expand Up @@ -239,11 +239,13 @@ jinja2==3.1.4
# via
# code-annotations
# coreschema
lxml[html-clean,html_clean]==5.1.1
lxml[html-clean,html_clean]==5.3.0
# via
# -c requirements/constraints.txt
# edx-credentials-themes
# edx-i18n-tools
# lxml-html-clean
lxml-html-clean==0.4.1
# via lxml
markdown==3.7
# via -r requirements/base.in
markupsafe==3.0.2
Expand Down
5 changes: 0 additions & 5 deletions requirements/constraints.txt
Original file line number Diff line number Diff line change
Expand Up @@ -11,11 +11,6 @@
# Common constraints for edx repos
-c common_constraints.txt

# Pinning lxml to < 5.2 as edx-i18n-tools package needs to be updated.
# Release notes: https://pypi.org/project/lxml/5.2.0/
# Github issue: https://github.com/openedx/i18n-tools/issues/144
lxml<5.2

# Pinning edx-django-utils to <6
# v6 drops support for python versions <3.12
# Changelog: https://github.com/openedx/edx-django-utils/blob/master/CHANGELOG.rst#600---2024-10-09
Expand Down
8 changes: 6 additions & 2 deletions requirements/dev.txt
Original file line number Diff line number Diff line change
Expand Up @@ -372,12 +372,16 @@ jinja2==3.1.4
# -r requirements/test.txt
# code-annotations
# coreschema
lxml[html-clean]==5.1.1
lxml[html-clean]==5.3.0
# via
# -c requirements/constraints.txt
# -r requirements/test.txt
# edx-credentials-themes
# edx-i18n-tools
# lxml-html-clean
lxml-html-clean==0.4.1
# via
# -r requirements/test.txt
# lxml
markdown==3.7
# via -r requirements/test.txt
markupsafe==3.0.2
Expand Down
12 changes: 8 additions & 4 deletions requirements/production.txt
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,9 @@ backoff==2.2.1
# segment-analytics-python
bleach==6.2.0
# via -r requirements/base.txt
boto3==1.35.73
boto3==1.35.74
# via django-ses
botocore==1.35.73
botocore==1.35.74
# via
# boto3
# s3transfer
Expand Down Expand Up @@ -321,12 +321,16 @@ jmespath==1.0.1
# via
# boto3
# botocore
lxml[html-clean]==5.1.1
lxml[html-clean]==5.3.0
# via
# -c requirements/constraints.txt
# -r requirements/base.txt
# edx-credentials-themes
# edx-i18n-tools
# lxml-html-clean
lxml-html-clean==0.4.1
# via
# -r requirements/base.txt
# lxml
markdown==3.7
# via -r requirements/base.txt
markupsafe==3.0.2
Expand Down
8 changes: 6 additions & 2 deletions requirements/test.txt
Original file line number Diff line number Diff line change
Expand Up @@ -345,12 +345,16 @@ jinja2==3.1.4
# -r requirements/base.txt
# code-annotations
# coreschema
lxml[html-clean]==5.1.1
lxml[html-clean]==5.3.0
# via
# -c requirements/constraints.txt
# -r requirements/base.txt
# edx-credentials-themes
# edx-i18n-tools
# lxml-html-clean
lxml-html-clean==0.4.1
# via
# -r requirements/base.txt
# lxml
markdown==3.7
# via -r requirements/base.txt
markupsafe==3.0.2
Expand Down
6 changes: 4 additions & 2 deletions requirements/translations.txt
Original file line number Diff line number Diff line change
Expand Up @@ -12,10 +12,12 @@ django==4.2.16
# edx-i18n-tools
edx-i18n-tools==1.6.3
# via -r requirements/translations.in
lxml[html-clean,html_clean]==5.1.1
lxml[html-clean,html_clean]==5.3.0
# via
# -c requirements/constraints.txt
# edx-i18n-tools
# lxml-html-clean
lxml-html-clean==0.4.1
# via lxml
path==16.16.0
# via edx-i18n-tools
polib==1.2.0
Expand Down

0 comments on commit cbdc38b

Please sign in to comment.