Skip to content
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.

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

feat: updated to python 3.12 #118

Merged
merged 6 commits into from
Nov 15, 2024
Merged
Show file tree
Hide file tree
Changes from 5 commits
Commits
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
4 changes: 2 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ jobs:
strategy:
matrix:
os: [ubuntu-20.04]
python-version: ['3.11']
python-version: ['3.12']
toxenv: [quality, pii_check, django42]

steps:
Expand All @@ -37,7 +37,7 @@ jobs:
run: tox

- name: Run coverage
if: matrix.python-version == '3.11' && matrix.toxenv == 'django42'
if: matrix.python-version == '3.12' && matrix.toxenv == 'django42'
uses: py-cov-action/python-coverage-comment-action@v3
with:
GITHUB_TOKEN: ${{ github.token }}
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/pypi-publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ jobs:
- name: setup python
uses: actions/setup-python@v2
with:
python-version: 3.8
python-version: 3.12

- name: Install pip
run: pip install -r requirements/pip.txt
Expand Down
4 changes: 4 additions & 0 deletions CHANGELOG.rst
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,10 @@ Unreleased
**********
* Add LearningAssistantMessage model

4.4.0 - 2024-10-25
******************
* Upgraded to use ``Python 3.12``

4.3.3 - 2024-10-15
******************
* Use `LEARNING_ASSISTANT_PROMPT_TEMPLATE` for prompt
Expand Down
1 change: 0 additions & 1 deletion learning_assistant/plugins.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
"""
Plugins for the Learning Assistant application.
"""
# pylint: disable=import-error
from openedx.core.djangoapps.course_apps.plugins import CourseApp

from learning_assistant import plugins_api
Expand Down
2 changes: 1 addition & 1 deletion learning_assistant/text_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ def cleanup_text(text):
return stripped


class _HTMLToTextHelper(HTMLParser): # lint-amnesty, pylint: disable=abstract-method
class _HTMLToTextHelper(HTMLParser):
"""
Helper function for html_to_text below.
"""
Expand Down
13 changes: 8 additions & 5 deletions pylintrc
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
# ** DO NOT EDIT THIS FILE **
# ***************************
#
# This file was generated by edx-lint: https://github.com/edx/edx-lint
# This file was generated by edx-lint: https://github.com/openedx/edx-lint
#
# If you want to change this file, you have two choices, depending on whether
# you want to make a local change that applies only to this repo, or whether
Expand All @@ -28,7 +28,7 @@
# CENTRAL CHANGE:
#
# 1. Edit the pylintrc file in the edx-lint repo at
# https://github.com/edx/edx-lint/blob/master/edx_lint/files/pylintrc
# https://github.com/openedx/edx-lint/blob/master/edx_lint/files/pylintrc
#
# 2. install the updated version of edx-lint (in edx-lint):
#
Expand Down Expand Up @@ -64,7 +64,7 @@
# SERIOUSLY.
#
# ------------------------------
# Generated by edx-lint version: 5.2.4
# Generated by edx-lint version: 5.4.0
# ------------------------------
[MASTER]
ignore = migrations
Expand Down Expand Up @@ -259,6 +259,7 @@ enable =
useless-suppression,
disable =
bad-indentation,
broad-exception-raised,
consider-using-f-string,
duplicate-code,
file-ignored,
Expand Down Expand Up @@ -290,6 +291,8 @@ disable =
django-not-configured,
consider-using-with,
bad-option-value,
import-error,
too-many-positional-arguments,

[REPORTS]
output-format = text
Expand Down Expand Up @@ -384,6 +387,6 @@ ext-import-graph =
int-import-graph =

[EXCEPTIONS]
overgeneral-exceptions = Exception
overgeneral-exceptions = builtins.Exception

# d103a3939f8137bacf02447b6f210552aa7f2827
# f8335666f17965c1c4bba3fc53bf25a343eaf3bf
2 changes: 2 additions & 0 deletions pylintrc_tweaks
Original file line number Diff line number Diff line change
Expand Up @@ -9,3 +9,5 @@ disable+=
django-not-configured,
consider-using-with,
bad-option-value,
import-error,
too-many-positional-arguments,
33 changes: 15 additions & 18 deletions requirements/base.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
#
# This file is autogenerated by pip-compile with Python 3.11
# This file is autogenerated by pip-compile with Python 3.12
# by the following command:
#
# make upgrade
Expand All @@ -10,15 +10,15 @@ attrs==24.2.0
# via -r requirements/base.in
certifi==2024.8.30
# via requests
cffi==1.17.0
cffi==1.17.1
# via
# cryptography
# pynacl
charset-normalizer==3.3.2
charset-normalizer==3.4.0
# via requests
click==8.1.7
# via edx-django-utils
cryptography==43.0.0
cryptography==43.0.3
# via pyjwt
django==4.2.16
# via
Expand All @@ -33,7 +33,7 @@ django==4.2.16
# edx-drf-extensions
django-crum==0.7.9
# via edx-django-utils
django-model-utils==4.5.1
django-model-utils==5.0.0
# via -r requirements/base.in
django-waffle==4.1.0
# via
Expand All @@ -44,33 +44,33 @@ djangorestframework==3.15.2
# -r requirements/base.in
# drf-jwt
# edx-drf-extensions
dnspython==2.6.1
dnspython==2.7.0
# via pymongo
drf-jwt==1.19.2
# via edx-drf-extensions
edx-django-utils==5.15.0
edx-django-utils==7.0.0
# via
# edx-drf-extensions
# edx-rest-api-client
edx-drf-extensions==10.4.0
# via -r requirements/base.in
edx-opaque-keys==2.10.0
edx-opaque-keys==2.11.0
# via
# -r requirements/base.in
# edx-drf-extensions
edx-rest-api-client==5.7.1
edx-rest-api-client==6.0.0
# via -r requirements/base.in
idna==3.8
idna==3.10
# via requests
jinja2==3.1.4
# via -r requirements/base.in
markupsafe==2.1.5
markupsafe==3.0.2
# via jinja2
newrelic==9.13.0
newrelic==10.2.0
# via edx-django-utils
pbr==6.1.0
# via stevedore
psutil==6.0.0
psutil==6.1.0
# via edx-django-utils
pycparser==2.22
# via cffi
Expand All @@ -79,19 +79,16 @@ pyjwt[crypto]==2.9.0
# drf-jwt
# edx-drf-extensions
# edx-rest-api-client
pymongo==4.8.0
pymongo==4.10.1
# via edx-opaque-keys
pynacl==1.5.0
# via edx-django-utils
requests==2.32.3
# via
# edx-drf-extensions
# edx-rest-api-client
# slumber
semantic-version==2.10.0
# via edx-drf-extensions
slumber==0.7.1
# via edx-rest-api-client
sqlparse==0.5.1
# via django
stevedore==5.3.0
Expand All @@ -100,5 +97,5 @@ stevedore==5.3.0
# edx-opaque-keys
typing-extensions==4.12.2
# via edx-opaque-keys
urllib3==2.2.2
urllib3==2.2.3
# via requests
14 changes: 7 additions & 7 deletions requirements/ci.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
#
# This file is autogenerated by pip-compile with Python 3.11
# This file is autogenerated by pip-compile with Python 3.12
# by the following command:
#
# make upgrade
Expand All @@ -10,25 +10,25 @@ chardet==5.2.0
# via tox
colorama==0.4.6
# via tox
distlib==0.3.8
distlib==0.3.9
# via virtualenv
filelock==3.15.4
filelock==3.16.1
# via
# tox
# virtualenv
packaging==24.1
# via
# pyproject-api
# tox
platformdirs==4.2.2
platformdirs==4.3.6
# via
# tox
# virtualenv
pluggy==1.5.0
# via tox
pyproject-api==1.7.1
pyproject-api==1.8.0
# via tox
tox==4.18.0
tox==4.23.2
# via -r requirements/ci.in
virtualenv==20.26.3
virtualenv==20.27.0
# via tox
Loading
Loading