Skip to content

Commit

Permalink
feat: adding python3.11 support.
Browse files Browse the repository at this point in the history
  • Loading branch information
awais786 committed Apr 17, 2024
1 parent 9f60a97 commit d9af1b8
Show file tree
Hide file tree
Showing 6 changed files with 11 additions and 14 deletions.
9 changes: 6 additions & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,11 +9,14 @@ on:
jobs:
quality_and_translations_tests:
runs-on: ubuntu-latest
strategy:
matrix:
python-version: ["3.8", "3.11"]
steps:
- uses: actions/checkout@v4
- uses: actions/setup-python@v5
with:
python-version: "3.8"
python-version: ${{ matrix.python-version }}
architecture: x64
- name: Install Dependencies
run: make requirements
Expand All @@ -33,13 +36,13 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
python-version: ["py38"]
python-version: ["3.8", "3.11"]
django-version: ["django42"]
steps:
- uses: actions/checkout@v4
- uses: actions/setup-python@v5
with:
python-version: "3.8"
python-version: ${{ matrix.python-version }}
architecture: x64
- name: Setup Nodejs Env
run: echo "NODE_VER=`cat .nvmrc`" >> $GITHUB_ENV
Expand Down
2 changes: 1 addition & 1 deletion credentials/apps/records/tests/test_views.py
Original file line number Diff line number Diff line change
Expand Up @@ -196,7 +196,7 @@ def test_email_content_complete(self):
# Check output and make sure it seems correct
self.assertEqual(len(mail.outbox), 1)
email = mail.outbox[0]
message = str(email.message())
message = email.Message.as_string()
self.assertIn(self.program.title + " Credit Request for", email.subject)
self.assertIn(
self.user.get_full_name() + " would like to apply for credit in the " + self.pathway.name, message
Expand Down
4 changes: 0 additions & 4 deletions requirements/constraints.txt
Original file line number Diff line number Diff line change
Expand Up @@ -11,10 +11,6 @@
# Common constraints for edx repos
-c common_constraints.txt

# The update to pyyaml 6.x failed as docker-compose wants <6,>=3.10. Pinning to <6.0. This constraint will be
# re-evaluated as part of APER-1556.
pyyaml<6.0

# Pinning urllib3 to versions < 2.x as this conflicts with boto. This constraint will be re-evaluated as part of
# APER-2422
urllib3<2
Expand Down
2 changes: 0 additions & 2 deletions requirements/dev.in
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,6 @@
# i18n
edx-i18n-tools

# for docker devstack
docker-compose

# More efficient checking for runserver reload trigger events
pywatchman; "linux" in sys_platform
Expand Down
6 changes: 3 additions & 3 deletions requirements/test.txt
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ backoff==2.2.1
# segment-analytics-python
backports-zoneinfo==0.2.1
# via
# -c requirements/constraints.txt
# -r requirements/base.txt
# django
# djangorestframework
Expand Down Expand Up @@ -256,7 +257,7 @@ jinja2==3.1.3
# -r requirements/base.txt
# code-annotations
# coreschema
lxml[html-clean]==5.1.1
lxml[html-clean]==5.1.0
# via
# -c requirements/constraints.txt
# -r requirements/base.txt
Expand Down Expand Up @@ -402,9 +403,8 @@ pytz==2024.1
# via
# -r requirements/base.txt
# drf-yasg
pyyaml==5.4.1
pyyaml==6.0.1
# via
# -c requirements/constraints.txt
# -r requirements/base.txt
# code-annotations
# drf-yasg
Expand Down
2 changes: 1 addition & 1 deletion tox.ini
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
[tox]
envlist = py38-django{42}
envlist = py{3.8,3.11}-django{42}
skipsdist = true

[pytest]
Expand Down

0 comments on commit d9af1b8

Please sign in to comment.