edina / nbexchange

External exchange for nbgrader
Other
6 stars 2 forks source link

Update to python 3.11 #125

Closed perllaghu closed 1 year ago

perllaghu commented 1 year ago

Currently put here for review.... I'm wondering about going through the whole code-base and, for example, checking all strings are f-strings, rather than having a mixture for formatted-string constructs

codecov[bot] commented 1 year ago

Codecov Report

Merging #125 (c7603f9) into master (26515f9) will increase coverage by 0.00%. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #125   +/-   ##
=======================================
  Coverage   88.46%   88.47%           
=======================================
  Files          72       73    +1     
  Lines        6276     6280    +4     
=======================================
+ Hits         5552     5556    +4     
  Misses        724      724           
Impacted Files Coverage Δ
.../alembic/versions/20190402_add_collected_action.py 83.33% <ø> (ø)
...ns/6f2a6c00affb_add_feedback_to_ation_enum_type.py 65.71% <ø> (ø)
...versions/d500457efb3b_create_nb_exchange_tables.py 51.85% <ø> (ø)
...s/f3345539f08d_change_assignment_name_data_type.py 66.66% <ø> (ø)
nbexchange/handlers/assignment.py 84.00% <ø> (ø)
nbexchange/handlers/base.py 98.92% <ø> (ø)
nbexchange/handlers/collection.py 89.52% <ø> (ø)
nbexchange/handlers/submission.py 84.33% <ø> (ø)
nbexchange/models/feedback.py 91.83% <ø> (ø)
nbexchange/models/notebooks.py 92.10% <ø> (ø)
... and 26 more

... and 1 file with indirect coverage changes

perllaghu commented 1 year ago

Updated to use the latest [mostly] lab-compatible nbgrader code-base.

I've not gone through the code checking for consistency or anything... that should probably be a separate issue on it's own.