From 2d78390f0217ca840641b423387b59bc8ae92fee Mon Sep 17 00:00:00 2001 From: Feanil Patel Date: Wed, 7 May 2025 14:38:16 -0400 Subject: [PATCH] chore: Upgrade type stubs and other related packages. Upgrade typing related packages and update to ignore one new annotation complaint and drop and older complaint that is no longer relevant. --- .../core/djangoapps/content_libraries/api/libraries.py | 6 ++++-- requirements/edx-sandbox/base.txt | 2 +- requirements/edx/base.txt | 2 +- requirements/edx/development.txt | 8 ++++---- requirements/edx/doc.txt | 2 +- requirements/edx/testing.txt | 2 +- 6 files changed, 12 insertions(+), 10 deletions(-) diff --git a/openedx/core/djangoapps/content_libraries/api/libraries.py b/openedx/core/djangoapps/content_libraries/api/libraries.py index 8549060f3e..d09e59f379 100644 --- a/openedx/core/djangoapps/content_libraries/api/libraries.py +++ b/openedx/core/djangoapps/content_libraries/api/libraries.py @@ -323,8 +323,10 @@ def require_permission_for_library_key(library_key: LibraryLocatorV2, user: User Raises django.core.exceptions.PermissionDenied if the user doesn't have permission. """ - library_obj = ContentLibrary.objects.get_by_key(library_key) # type: ignore[attr-defined] - if not user.has_perm(permission, obj=library_obj): + library_obj = ContentLibrary.objects.get_by_key(library_key) + # obj should be able to read any valid model object but mypy thinks it can only be + # "User | AnonymousUser | None" + if not user.has_perm(permission, obj=library_obj): # type:ignore[arg-type] raise PermissionDenied return library_obj diff --git a/requirements/edx-sandbox/base.txt b/requirements/edx-sandbox/base.txt index 68754ed30d..6c864800f5 100644 --- a/requirements/edx-sandbox/base.txt +++ b/requirements/edx-sandbox/base.txt @@ -81,7 +81,7 @@ six==1.17.0 # via # codejail-includes # python-dateutil -sympy==1.13.3 +sympy==1.14.0 # via # -r requirements/edx-sandbox/base.in # openedx-calc diff --git a/requirements/edx/base.txt b/requirements/edx/base.txt index 190387cd66..da65bcf088 100644 --- a/requirements/edx/base.txt +++ b/requirements/edx/base.txt @@ -1141,7 +1141,7 @@ stevedore==5.4.1 # edx-opaque-keys super-csv==4.1.0 # via edx-bulk-grades -sympy==1.13.3 +sympy==1.14.0 # via openedx-calc testfixtures==8.3.0 # via edx-enterprise diff --git a/requirements/edx/development.txt b/requirements/edx/development.txt index 6b778fc216..91665ae224 100644 --- a/requirements/edx/development.txt +++ b/requirements/edx/development.txt @@ -573,12 +573,12 @@ django-storages==1.14.3 # -r requirements/edx/doc.txt # -r requirements/edx/testing.txt # edxval -django-stubs==5.1.3 +django-stubs==5.2.0 # via # -c requirements/edx/../constraints.txt # -r requirements/edx/development.in # djangorestframework-stubs -django-stubs-ext==5.1.3 +django-stubs-ext==5.2.0 # via django-stubs django-user-tasks==3.3.0 # via @@ -621,7 +621,7 @@ djangorestframework==3.14.0 # openedx-learning # ora2 # super-csv -djangorestframework-stubs==3.15.3 +djangorestframework-stubs==3.16.0 # via -r requirements/edx/development.in djangorestframework-xml==2.0.0 # via @@ -2060,7 +2060,7 @@ super-csv==4.1.0 # -r requirements/edx/doc.txt # -r requirements/edx/testing.txt # edx-bulk-grades -sympy==1.13.3 +sympy==1.14.0 # via # -r requirements/edx/doc.txt # -r requirements/edx/testing.txt diff --git a/requirements/edx/doc.txt b/requirements/edx/doc.txt index 95c5d24995..8a4f0affbb 100644 --- a/requirements/edx/doc.txt +++ b/requirements/edx/doc.txt @@ -1437,7 +1437,7 @@ super-csv==4.1.0 # via # -r requirements/edx/base.txt # edx-bulk-grades -sympy==1.13.3 +sympy==1.14.0 # via # -r requirements/edx/base.txt # openedx-calc diff --git a/requirements/edx/testing.txt b/requirements/edx/testing.txt index d6d2f9f912..f6fe4fa3f2 100644 --- a/requirements/edx/testing.txt +++ b/requirements/edx/testing.txt @@ -1525,7 +1525,7 @@ super-csv==4.1.0 # via # -r requirements/edx/base.txt # edx-bulk-grades -sympy==1.13.3 +sympy==1.14.0 # via # -r requirements/edx/base.txt # openedx-calc