Skip to content

fix: upgrade ora2 dependency to version 6.17.2 and remove loremipsum d… #307

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.

Sign up for GitHub

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

Merged
alenkadev merged 8 commits into from
May 31, 2026
Merged

fix: upgrade ora2 dependency to version 6.17.2 and remove loremipsum d… #307

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
51fc613
fix: upgrade ora2 dependency to version 7.0.0 and remove loremipsum d…
May 27, 2026
ddf0675
Merge branch 'release-ulmo' into alenka/fixing-edx-platform-workflow
alenkadev May 27, 2026
074c8f0
fix: update ora2 dependency to version 7.0.0 and remove loremipsum de…
May 27, 2026
21364b1
fix: downgrade ora2 dependency to version 6.17.2 across multiple requ…
May 27, 2026
4de3bb5
Merge branch 'release-ulmo' into alenka/fixing-edx-platform-workflow
alenkadev May 28, 2026
c7d6ed2
fix: improve translation extraction process in Makefile
May 28, 2026
04f72b3
fix: streamline translation extraction process and downgrade ora2 dep…
May 29, 2026
3427f64
fix: enhance translation extraction process in Makefile
May 29, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 1 addition & 3 deletions requirements/edx/base.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -713,8 +713,6 @@ lazy==1.6
# lti-consumer-xblock
# ora2
# xblock
loremipsum==1.0.5
# via ora2
lti-consumer-xblock==9.14.2
Comment thread
alenkadev marked this conversation as resolved.
# via -r requirements/edx/kernel.in
lxml[html-clean]==5.3.2
Expand Down Expand Up @@ -845,7 +843,7 @@ openedx-learning==0.27.1
# -r requirements/edx/kernel.in
optimizely-sdk==5.2.0
# via -r requirements/edx/bundled.in
ora2==6.16.4
ora2==6.17.2

@michaelroytman michaelroytman May 29, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hm. Copilot is right that bundle.in is included in base.in, but I assume that the override here is good enough, especially because this is how the requirements are written in openedx/edx-platform.

If this doesn't fix it and we still run into issues, we could consider addressing this issue and contributing the change upstream.

@alenkadev alenkadev May 31, 2026

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for catching this. bundled.in is included via base.in, so I agree there is a theoretical possibility of future resolves selecting a different version. However, this override pattern matches the current approach used in upstream edx-platform, and the explicit pin in base.txt ensures the compiled requirements use ora2==6.17.2. Given that, I'd prefer to keep this aligned with upstream for now and revisit updating the source constraint if we continue to see dependency resolution issues.

# via -r requirements/edx/bundled.in
packaging==25.0
# via
Expand Down
7 changes: 1 addition & 6 deletions requirements/edx/development.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1187,11 +1187,6 @@ libsass==0.10.0
# via
# -c requirements/constraints.txt
# -r requirements/edx/assets.txt
loremipsum==1.0.5
# via
# -r requirements/edx/doc.txt
# -r requirements/edx/testing.txt
# ora2
lti-consumer-xblock==9.14.2
# via
# -r requirements/edx/doc.txt
Expand Down Expand Up @@ -1400,7 +1395,7 @@ optimizely-sdk==5.2.0
# via
# -r requirements/edx/doc.txt
# -r requirements/edx/testing.txt
ora2==6.16.4
ora2==6.17.2
# via
# -r requirements/edx/doc.txt
# -r requirements/edx/testing.txt
Expand Down
6 changes: 1 addition & 5 deletions requirements/edx/doc.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -865,10 +865,6 @@ lazy==1.6
# lti-consumer-xblock
# ora2
# xblock
loremipsum==1.0.5
# via
# -r requirements/edx/base.txt
# ora2
lti-consumer-xblock==9.14.2
# via -r requirements/edx/base.txt
lxml[html-clean]==5.3.2
Expand Down Expand Up @@ -1019,7 +1015,7 @@ openedx-learning==0.27.1
# -r requirements/edx/base.txt
optimizely-sdk==5.2.0
# via -r requirements/edx/base.txt
ora2==6.16.4
ora2==6.17.2
# via -r requirements/edx/base.txt
packaging==25.0
# via
Expand Down
6 changes: 1 addition & 5 deletions requirements/edx/testing.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -906,10 +906,6 @@ lazy==1.6
# lti-consumer-xblock
# ora2
# xblock
loremipsum==1.0.5
# via
# -r requirements/edx/base.txt
# ora2
lti-consumer-xblock==9.14.2
# via -r requirements/edx/base.txt
lxml[html-clean]==5.3.2
Expand Down Expand Up @@ -1064,7 +1060,7 @@ openedx-learning==0.27.1
# -r requirements/edx/base.txt
optimizely-sdk==5.2.0
# via -r requirements/edx/base.txt
ora2==6.16.4
ora2==6.17.2
# via -r requirements/edx/base.txt
packaging==25.0
# via
Expand Down
Loading