Skip to content

Commit

Permalink
build: Upgrade django-stubs even further, to fix mypy
Browse files Browse the repository at this point in the history
Rather than constraining django-stubs' major version to our django
major version (4.x.x), we are going to go one ahead (5.x.x), as
recommended by python/mypy#17958

Also includes an unrelated common_constraints update.
  • Loading branch information
kdmccormick committed Jan 9, 2025
1 parent f7ccd7c commit 9262c9a
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 10 deletions.
8 changes: 5 additions & 3 deletions requirements/constraints.txt
Original file line number Diff line number Diff line change
Expand Up @@ -61,11 +61,13 @@ django-webpack-loader==0.7.0
djangorestframework<3.15.0

# Date: 2024-07-19
# Generally speaking, the major version of django-stubs should match the major version of django.
# Specifically, we need to perpetually constrain django-stubs to a compatible version based on:
# Generally speaking, the major version of django-stubs must either match the major version
# of django, or exceed it by 1. So, we will need to perpetually constrain django-stubs and
# update it as we perform django upgrades. For more details, see:
# https://github.com/typeddjango/django-stubs?tab=readme-ov-file#version-compatibility
# including the note on "Partial Support".
# Issue: https://github.com/openedx/edx-platform/issues/35275
django-stubs<5
django-stubs<6

# Date: 2024-07-23
# django-storages==1.14.4 breaks course imports
Expand Down
11 changes: 4 additions & 7 deletions requirements/edx/development.txt
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,7 @@ asgiref==3.8.1
# django
# django-cors-headers
# django-countries
# django-stubs
asn1crypto==1.5.1
# via
# -r requirements/edx/doc.txt
Expand Down Expand Up @@ -577,7 +578,7 @@ django-storages==1.14.3
# -r requirements/edx/doc.txt
# -r requirements/edx/testing.txt
# edxval
django-stubs==4.2.7
django-stubs==5.1.1
# via
# -c requirements/edx/../constraints.txt
# -r requirements/edx/development.in
Expand Down Expand Up @@ -625,7 +626,7 @@ djangorestframework==3.14.0
# openedx-learning
# ora2
# super-csv
djangorestframework-stubs==3.14.5
djangorestframework-stubs==3.15.2
# via -r requirements/edx/development.in
djangorestframework-xml==2.0.0
# via
Expand Down Expand Up @@ -1293,9 +1294,7 @@ multidict==6.1.0
# aiohttp
# yarl
mypy==1.14.1
# via
# -c requirements/edx/../constraints.txt
# -r requirements/edx/development.in
# via -r requirements/edx/development.in
mypy-extensions==1.0.0
# via mypy
mysqlclient==2.2.6
Expand Down Expand Up @@ -2120,8 +2119,6 @@ tqdm==4.67.1
# -r requirements/edx/testing.txt
# nltk
# openai
types-pytz==2024.2.0.20241221
# via django-stubs
types-pyyaml==6.0.12.20241230
# via
# django-stubs
Expand Down

0 comments on commit 9262c9a

Please sign in to comment.