openedx / axim-engineering

GitHub Issue repository for the Axim engineering team
https://openedx.atlassian.net/wiki/spaces/COMM/pages/3241640370/Axim+Collaborative+Engineering+Team
4 stars 3 forks source link

[GH Request] Remove required checks from auth-backends and edx-django-utils #1247

Closed huniafatima-arbi closed 2 months ago

huniafatima-arbi commented 2 months ago

Repository

auth-backends, edx-django-utils

Urgency

Medium (< 2 weeks)

Requested Change

Need to remove required checks of py38 from auth-backends and edx-django-utils repo in order to merge Python 3.8 support DEPR PRs. PR links are mentioned below

Reasoning

We are dropping the support of Python 3.8 as Python 3.8 end of life is Oct 2204. We need to upgrade to Python 3.12 before that to avoid breaking anything.

github-actions[bot] commented 2 months ago

Thank you for your report! @openedx/axim-oncall will triage within a business day. Simple requests usually take 2-3 business days to resolve; more complex requests could take longer.

brian-smith-tcril commented 2 months ago

@huniafatima-arbi I have removed the required checks and I have a question. On auth-backends I see both 3.11 and 3.12 tests in CI now - which should I add required checks for? Both? Just 3.12?

huniafatima-arbi commented 2 months ago

@brian-smith-tcril we will be adding the required checks for just 3.12.

brian-smith-tcril commented 2 months ago

I've added the required checks for 3.12 on auth-backends. Please let me know if you run into any issues or if there's anything I missed and I can reopen this!