Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

Python Requirements Update #690

Merged
merged 2 commits into from
Oct 31, 2023

Conversation

edx-requirements-bot
Copy link
Contributor

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

Deleted obsolete pull_requests:
#689

@edx-requirements-bot
Copy link
Contributor Author

List of packages in the PR without any issue.

  • babel changes from 2.13.0 to 2.13.1
  • boto3 changes from 1.28.67 to 1.28.73
  • botocore changes from 1.31.67 to 1.31.73
  • charset-normalizer changes from 3.3.0 to 3.3.1
  • cryptography changes from 41.0.4 to 41.0.5
  • faker changes from 19.11.0 to 19.12.0
  • filelock changes from 3.12.4 to 3.13.0
  • greenlet changes from 3.0.0 to 3.0.1
  • pip changes from 23.3 to 23.3.1
  • pydata-sphinx-theme changes from 0.14.1 to 0.14.2
  • pytest changes from 7.4.2 to 7.4.3
  • virtualenv changes from 20.24.5 to 20.24.6

@edx-requirements-bot
Copy link
Contributor Author

These Packages need manual review..

  • [MAJOR] djangorestframework-csv changes from 2.1.1 to 3.0.1
  • [REMOVED] unicodecsv (0.14.1) removed from the requirements

@codecov
Copy link

codecov bot commented Oct 27, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (09a8301) 97.51% compared to head (f4e5823) 97.51%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #690   +/-   ##
=======================================
  Coverage   97.51%   97.51%           
=======================================
  Files          53       53           
  Lines        3023     3023           
  Branches      266      266           
=======================================
  Hits         2948     2948           
  Misses         63       63           
  Partials       12       12           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@MichaelRoytman MichaelRoytman force-pushed the jenkins/upgrade-python-requirements-09a8301 branch 3 times, most recently from 780624d to 3c9d63a Compare October 30, 2023 13:36
False was the default value for navigation_with_keys. However, in version 0.14.2 of pydata-sphinx-theme, this default was removed and a warning was added that would be emitted whenever navigation_with_keys was not set. Because of the "SPHINXOPTS = -W" configuration in tox.ini, all warnings are promoted to an error. Therefore, it's necesary to set this value. I have set it to the default value explicitly.

Please see the following GitHub comments for context.
* pydata/pydata-sphinx-theme#1539
* pydata/pydata-sphinx-theme#987 (comment)
@MichaelRoytman MichaelRoytman force-pushed the jenkins/upgrade-python-requirements-09a8301 branch from 054d560 to f4e5823 Compare October 30, 2023 20:19
Copy link
Contributor

@MichaelRoytman MichaelRoytman left a comment

Choose a reason for hiding this comment

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

I reviewed these changes, and they look good to me.

I'm going to get a second review from a coworker to review my fixes.

Copy link
Contributor

@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.

👍

@MichaelRoytman MichaelRoytman merged commit 4123349 into master Oct 31, 2023
7 checks passed
@MichaelRoytman MichaelRoytman deleted the jenkins/upgrade-python-requirements-09a8301 branch October 31, 2023 13:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants