openedx / edx-lint

Custom tooling for pylint and other repo management tools
Apache License 2.0
50 stars 26 forks source link

chore: update Django constraint to match latest LTS version #403

Closed mariajgrimaldi closed 6 months ago

mariajgrimaldi commented 6 months ago

Description:

This PR updates the Django common constraint used by other projects' repositories to use the latest LTS version: https://openedx.atlassian.net/wiki/spaces/AC/pages/3767926815/Django+4.2+Upgrade+Plan

Merge checklist:

openedx-webhooks commented 6 months ago

Thanks for the pull request, @mariajgrimaldi! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

mphilbrick211 commented 6 months ago

@feanil - is this something Axim should review?

feanil commented 6 months ago

Yea, there are still 2 services that are not running django 4 so we'll need to update the local constraints for those services so we can merge this, I'll reach out to the team that's doing the upgrades for those final 2 services.

openedx-webhooks commented 6 months ago

@mariajgrimaldi 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.