Skip to content

Conversation

@muhammad-ammar
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings December 8, 2025 05:38
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 Python package dependencies across all requirements files, including minor and patch version updates for most packages along with a Sphinx major version upgrade for documentation.

  • Updates AWS SDK packages (boto3, botocore, s3transfer) across all environments
  • Upgrades Django from 4.2.26 to 4.2.27 (security/bug fix release)
  • Updates Sphinx documentation tool from 8.2.3 to 9.0.4 (major version upgrade)
  • Includes package rename from roman-numerals-py to roman-numerals in documentation dependencies

Reviewed changes

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

Show a summary per file
File Description
requirements/base.txt Updates core dependencies including boto3 (1.42.4), django (4.2.27), django-countries (8.2.0), and other base packages
requirements/dev.txt Mirrors base.txt updates for development environment dependencies
requirements/django.txt Updates Django to version 4.2.27
requirements/doc.txt Updates documentation dependencies including Sphinx major upgrade (9.0.4), beautifulsoup4, docutils, and package rename from roman-numerals-py to roman-numerals
requirements/production.txt Updates production dependencies including greenlet (3.3.0) and mirrors base package updates
requirements/test.txt Updates testing dependencies including pytest (9.0.2) and mirrors base package updates
requirements/tox.txt Updates platformdirs to version 4.5.1 for tox testing environment

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

# edx-rest-api-client
# sphinx
roman-numerals-py==3.1.0
roman-numerals==3.1.0
Copy link

Copilot AI Dec 8, 2025

Choose a reason for hiding this comment

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

The package name has changed from roman-numerals-py to roman-numerals. While this appears to be an intentional package rename (likely by the package maintainer), please verify that this is the correct package and that it's compatible with Sphinx 9.0.4, especially since both changes occur together in this file.

Copilot uses AI. Check for mistakes.
Copy link
Member Author

Choose a reason for hiding this comment

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

AA-Turner/roman-numerals@cc8ec3a

The roman-numerals-py package has been renamed to roman-numerals.

@github-actions
Copy link

github-actions bot commented Dec 8, 2025

Coverage report

This PR does not seem to contain any modification to coverable code.

@pwnage101 pwnage101 merged commit 864492c into master Dec 8, 2025
11 checks passed
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