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

chore: update actions and test with python 311 #2

Draft
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

MaferMazu
Copy link
Collaborator

@MaferMazu MaferMazu commented Nov 1, 2024

Description

This PR adds support to Python 3.8 and 3.11, drops support for the previous version of Python, and fixes the issues related to this support.

Changes

How to test

Code

  • All the checks from Github are passing.

Integration

Environment

  • Have a tutor environment with Tutor v18.1.4
  • Install codejail. You can follow the installation and usage process from the tutor-contrib-codejail README.
  • Perform a dev launch

Data

  • Import the MIT demo course running:
tutor dev do importdemocourse -r https://github.com/mitodl/mitx-grading-library -d course -v master

For more info: tutor dev do importdemocourse -h

  • Create an admin user
tutor dev do createuser --staff --superuser admin admin@example.com

Test

  1. Enter in Studio > MITx Grading Library > Content > Files
  2. Remove the python_lib.zip and add this one:
    python_lib.zip

Note

If you want to create your python_lib from this version, you can clone the repo, enter in it, and then:

find mitxgraders voluptuous -type f ! -path "*/__pycache__/*" | zip python_lib.zip -@
  1. Make sure the units are rendered and worked well.

Doc

Screencasts

I test the StringGraders, MatrixGraders, and SumGraders in the following video:

Screencast.from.2024-11-07.18-23-57.webm

Things checked

  • Update basic doc
  • Test doc in local
  • Test in an environment

Following work

  • Create a PR to remove the Python 2 exceptions and considerations from the code.
  • Bump version to 3.0.0 because we have breaking changes and edited the doc.

@MaferMazu MaferMazu force-pushed the mfmz/support-python-311 branch from 1026571 to 9f97cd8 Compare November 1, 2024 20:04
@MaferMazu MaferMazu force-pushed the mfmz/support-python-311 branch from 9f97cd8 to a8af6ef Compare November 1, 2024 22:06
@MaferMazu MaferMazu force-pushed the mfmz/support-python-311 branch from a8af6ef to 2f650ae Compare November 6, 2024 03:50
@MaferMazu MaferMazu force-pushed the mfmz/support-python-311 branch from 8d21d9a to 62abec1 Compare November 14, 2024 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant