Skip to content

Conversation

@ktyagiapphelix2u
Copy link

@ktyagiapphelix2u ktyagiapphelix2u commented Aug 7, 2025

Description

Remove the deprecated pyjwkest dependency

Jira Link

https://2u-internal.atlassian.net/browse/BOMS-89

Bumping Version PR

openedx/xblock-lti-consumer#572

Copilot AI review requested due to automatic review settings August 7, 2025 09:12
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 pull request removes the pyjwkest dependency from edx-exams as part of a broader effort to eliminate this package from the codebase. The PR updates all requirement files to reflect the removal of pyjwkest and associated dependencies that are no longer needed.

Key changes include:

  • Removal of pyjwkest==1.4.2 from all requirement files
  • Removal of future==1.0.0 dependency that was only needed by pyjwkest
  • Update of dependency paths to use relative paths instead of absolute paths
  • Version updates for various packages that are no longer constrained by removed dependencies

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated no comments.

Show a summary per file
File Description
requirements/validation.txt Removes pyjwkest and future dependencies, updates package versions and path references
requirements/test.txt Removes pyjwkest and future dependencies, updates package versions and path references
requirements/quality.txt Removes pyjwkest and future dependencies, updates package versions and path references
requirements/production.txt Removes pyjwkest and future dependencies, updates package versions and path references
requirements/pip.txt Updates pip version
requirements/pip-tools.txt Updates build and pip-tools versions
requirements/doc.txt Removes pyjwkest and future dependencies, updates package versions and path references

@github-actions
Copy link

github-actions bot commented Aug 7, 2025

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  edx_exams/apps/api/v1
  views.py
Project Total  

This report was generated by python-coverage-comment-action

@ktyagiapphelix2u ktyagiapphelix2u changed the title chore: removal of pyjwkest from edx-exams chore!: removal of pyjwkest from edx-exams Aug 11, 2025
Copy link

@jcapphelix jcapphelix left a comment

Choose a reason for hiding this comment

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

Looks good to go

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.

4 participants