Skip to content
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.

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

chore: Upgrade Python requirements #311

Merged
merged 3 commits into from
Aug 30, 2024

Conversation

edx-requirements-bot
Copy link
Contributor

Python requirements update. Please review the changelogs for the upgraded packages.

Copy link

github-actions bot commented Aug 27, 2024

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  edx_exams/apps/api/v1
  views.py
  edx_exams/apps/api/v1/tests
  test_views.py
  edx_exams/apps/core
  models.py
  statuses.py
  views.py
  edx_exams/apps/core/management/commands
  bulk_add_course_staff.py
  edx_exams/apps/core/management/commands/test
  test_bulk_add_course_staff.py
  edx_exams/apps/core/tests
  test_api.py
  test_context_processors.py
  test_email.py
  test_handlers.py
  test_views.py
  edx_exams/apps/lti
  views.py
  edx_exams/apps/lti/signals
  handlers.py
  edx_exams/apps/lti/signals/tests
  test_handlers.py
  edx_exams/apps/lti/tests
  test_views.py
  edx_exams/apps/router/tests
  test_interop.py
  test_views.py
Project Total  

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

@alangsto alangsto force-pushed the repo-tools/upgrade-python-requirements-513db6e branch 3 times, most recently from d3d751f to decb066 Compare August 30, 2024 14:08
@alangsto
Copy link
Member

importlib_resources used with Python < 3.9 needs to follow these instructions to avoid an import error: https://importlib-resources.readthedocs.io/en/latest/using.html#migrating-from-legacy.

To avoid this error and making changes to the importlib import, we can also update the github actions to use Python 3.9.

Copy link
Member

@schenedx schenedx left a comment

Choose a reason for hiding this comment

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

👍

@alangsto alangsto force-pushed the repo-tools/upgrade-python-requirements-513db6e branch from decb066 to 0fe67d3 Compare August 30, 2024 14:22
@alangsto alangsto merged commit 01b8d56 into main Aug 30, 2024
5 checks passed
@alangsto alangsto deleted the repo-tools/upgrade-python-requirements-513db6e branch August 30, 2024 14:27
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.

3 participants