Skip to content

chore: update to Python 3.11/3.12 and Django 4.2/5.2 #13

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Gi-ron wants to merge 6 commits into
base: main
Choose a base branch
from
Open

chore: update to Python 3.11/3.12 and Django 4.2/5.2 #13

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
9a8ec7e
feat: add support for Python 3.11/3.12 and Django 4.2/5.2
Gi-ron Mar 31, 2026
4e5392a
fix: commitlint workflow
Gi-ron Mar 31, 2026
5a56827
fix: update matrix ubuntu version
Gi-ron Mar 31, 2026
48b2e9d
feat: upgrade pip and setuptools in workflow
Gi-ron Mar 31, 2026
f5f9e56
feat: adding feedback pr
Gi-ron Apr 1, 2026
8fd3e1f
fix: update ci workflow with correct versions
Gi-ron Apr 1, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 13 additions & 7 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,21 +7,27 @@ on:
branches:
- '**'


jobs:
run_tests:
name: tests
runs-on: ${{ matrix.os }}
runs-on: ubuntu-24.04
strategy:
matrix:
os: [ubuntu-20.04]
python-version: ['3.8']
toxenv: [quality, django32, django40]
include:
- python-version: "3.11"
toxenv: "py311-django42"
- python-version: "3.11"
toxenv: "py311-django52"
- python-version: "3.12"
toxenv: "py312-django52"
- python-version: "3.11"
toxenv: "quality"

steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v4

- name: setup python
uses: actions/setup-python@v2
uses: actions/setup-python@v5
with:
python-version: ${{ matrix.python-version }}

Expand Down
23 changes: 19 additions & 4 deletions .github/workflows/commitlint.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,10 +1,25 @@
# Run commitlint on the commit messages in a pull request.

name: Lint Commit Messages

on:
- pull_request
pull_request:

defaults:
run:
shell: bash

jobs:
commitlint:
uses: edx/.github/.github/workflows/commitlint.yml@master
runs-on: ubuntu-latest
steps:
- name: Check out the repo
uses: actions/checkout@v4
with:
fetch-depth: 100

- name: Download configuration
Copy link
Copy Markdown

@magajh magajh Mar 31, 2026

Choose a reason for hiding this comment

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

@Gi-ron I’m curious why this step and the related changes in this action are needed now. Was the action not working properly before?

Copy link
Copy Markdown
Author

@Gi-ron Gi-ron Apr 1, 2026

Choose a reason for hiding this comment

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

The issue was that the default configuration file (downloaded from openedx/edx-lint) uses ES module syntax (export default), but the action expects CommonJS (module.exports). This caused a syntax error when the action tried to run. I’m not entirely sure if this failure was already present before (I didn’t run the action on previous commits)

Copy link
Copy Markdown

@magajh magajh Apr 1, 2026

Choose a reason for hiding this comment

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

@Gi-ron are we sure there is not a cleaner way to run this workflow?
If not, I'm ok with merging this but first I would like us to do a double check

run: |
wget --no-verbose -O commitlint.config.mjs \
https://raw.githubusercontent.com/openedx/edx-lint/HEAD/edx_lint/files/commitlint.config.js

- name: Run commitlint
uses: wagoid/commitlint-github-action@v6
6 changes: 3 additions & 3 deletions .github/workflows/pypi-publish.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,11 +11,11 @@ jobs:

steps:
- name: Checkout
uses: actions/checkout@v2
uses: actions/checkout@v4
- name: setup python
uses: actions/setup-python@v2
uses: actions/setup-python@v5
with:
python-version: 3.8
python-version: 3.11

- name: Install pip
run: pip install -r requirements/pip.txt
Expand Down
35 changes: 35 additions & 0 deletions CHANGELOG.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,41 @@ Change Log
Unreleased
~~~~~~~~~~

[1.0.0] - 2026-03-30
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Added
_____

* Support for Python 3.11 and 3.12.
* Support for Django 4.2 and 5.2 (compatible with Open edX Teak and Ulmo releases).
* New GitHub Actions test matrix with the supported Python/Django combinations.
* Set `DJANGO_SETTINGS_MODULE` in the `quality` tox environment to avoid pylint configuration errors.

Changed
_______

* Updated `social-auth-core` from 4.1.0 to 4.8.5.
* Upgraded all dependencies via `make upgrade`.
* Updated GitHub workflows (`ci.yml`, `pypi-publish.yml`) to use recent action versions.
* Updated `tox.ini` to test against the new Python and Django versions.
* Updated classifiers in `setup.py` to reflect Python 3.11/3.12 and Django 4.2/5.2.
* Updated `README.rst` compatibility table to include Teak.

Removed
_______

* Dropped support for Python 3.8.
* Dropped support for Django 2.2, 3.2, and 4.0.
* Dropped support for Open edX releases prior to Sumac.
* Removed obsolete pylint options (regenerated `.pylintrc` with edx-lint).

Fixed
_____

* Fixed pylint warnings in `setup.py` by using `with` statements for file operations.
* Fixed `tox.ini` to allow external commands in the `quality` environment.

[0.2.3] - 2022-08-11
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Expand Down
17 changes: 17 additions & 0 deletions README.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -195,3 +195,20 @@ For more information about these options, see the `Getting Help`_ page.
.. |license-badge| image:: https://img.shields.io/github/license/eduNEXT/openedx-cas.svg
:target: https://github.com/eduNEXT/openedx-cas/blob/main/LICENSE.txt
:alt: License

## Compatibility Notes
-------------------------

+---------------------+----------------+
| Open edX Release | Version |
+=====================+================+
| Redwood | <= 0.2.4 |
+---------------------+----------------+
| Sumac | >= 1.0.0 |
+---------------------+----------------+
| Teak | >= 1.0.0 |
+---------------------+----------------+
| Ulmo | >= 1.0.0 |
+---------------------+----------------+

**Note:** Version 0.3.0 and later are compatible with Sumac, Teak, and Ulmo. For releases Redwood and earlier, please use version 0.2.4 or any 0.2.x release.
2 changes: 1 addition & 1 deletion openedx_cas/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,6 @@
Support for Certralized Authentication System (CAS) in the Open edX authentication flows.
"""

__version__ = '0.2.4'
__version__ = '1.0.0'

default_app_config = 'openedx_cas.apps.OpenedxCasConfig' # pylint: disable=invalid-name
Loading
Loading