Skip to content

build: Upgrade to which removes loremipsum base dep#130

Open
ktyagiapphelix2u wants to merge 1 commit intorelease-ulmofrom
ktyagi/ora
Open

build: Upgrade to which removes loremipsum base dep#130
ktyagiapphelix2u wants to merge 1 commit intorelease-ulmofrom
ktyagi/ora

Conversation

@ktyagiapphelix2u
Copy link

No description provided.

Copilot AI review requested due to automatic review settings February 16, 2026 09:55
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR upgrades the ora2 (Open Response Assessment XBlock) package from version 6.16.4 to 6.17.2, which removes the loremipsum dependency as it is no longer needed. The upgrade simplifies the dependency tree by eliminating an unnecessary transitive dependency.

Changes:

  • Upgraded ora2 from 6.16.4 to 6.17.2 across all requirements files
  • Removed loremipsum==1.0.5 dependency (no longer required by ora2)

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
requirements/edx/base.txt Updated ora2 version and removed loremipsum dependency
requirements/edx/testing.txt Updated ora2 version and removed loremipsum dependency
requirements/edx/doc.txt Updated ora2 version and removed loremipsum dependency
requirements/edx/development.txt Updated ora2 version and removed loremipsum dependency

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

optimizely-sdk==5.2.0
# via -r requirements/edx/bundled.in
ora2==6.16.4
ora2==6.17.2
Copy link

Copilot AI Feb 16, 2026

Choose a reason for hiding this comment

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

The PR title appears to be incomplete: "build: Upgrade to which removes loremipsum base dep". It should specify the package name and version being upgraded to, such as "build: Upgrade to ora2==6.17.2 which removes loremipsum base dep".

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants