search

openedx / olxcleaner

Tool for checking edX courses for errors and creating content reports
GNU General Public License v3.0
3 stars 6 forks source link

Modernization and Python Upgrade #21

Closed feanil closed 6 months ago

feanil commented 6 months ago
feanil commented 6 months ago

@mariajgrimaldi I noticed that the requirements setup in this repo was not standard since it came from a fork initially. I've updated it, run make upgrade and added the testing for 3.11 and 3.12. Can you review the PR for me?

codecov[bot] commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.60%. Comparing base (0b14b27) to head (494ddc2).

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/openedx/olxcleaner/pull/21/graphs/tree.svg?width=650&height=150&src=pr&token=DF2HJTEPOP&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx)](https://app.codecov.io/gh/openedx/olxcleaner/pull/21?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx) ```diff @@ Coverage Diff @@ ## master #21 +/- ## ======================================= Coverage 99.60% 99.60% ======================================= Files 32 32 Lines 1251 1268 +17 ======================================= + Hits 1246 1263 +17 Misses 5 5 ```
mariajgrimaldi commented 6 months ago

@feanil: on it!