[GH Request] Remove checks from branch protection rules in Course Discovery

[iamsobanjaved]

Firm Name

2U

Urgency

Medium (< 2 weeks)

Requested Change

Kindly remove these checks from branch protection rules, in the course-discovery repo

• pytest (py38, django32, mysql57, 1)
• pytest (py38, django32, mysql57, 2)
• pytest (py38, django32, mysql57, 3)
• pytest (py38, django32, mysql57, 4)
• pytest (py38, django32, mysql57, 5)
• pytest (py38, django32, mysql57, 6)

Ref PR: openedx/course-discovery#4088

Reasoning

We want to merge Django upgrade PR, but these checks are stopping us. We can disable these for now, and enable Django 4.2 once we merge the upgrade PR

[openedx-workflow-automation]

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.

[kdmccormick]

Django 3.2 checks are no longer required. Just ping me on this ticket again when you want the Django 4.2 checks marked as required.

Here's what the checks looked like before, for our reference:

[kdmccormick]

@iamsobanjaved Should I make the Django 4.2 unit tests required now?

[iamsobanjaved]

We are yet testing the upgrade on stage and already hit an issue and made fix PR for that. I think we can wait till tomorrow, until the upgrade is complete.

[kdmccormick]

Sounds good, just let me know.

[iamsobanjaved]

@kdmccormick it is live on the production so you can now mark Django 4.2 tests as required.

[kdmccormick]

Done!