[GH Request] Remove checks from branch protection rules in edx-notes-api

[iamsobanjaved]

Firm Name

2U

Urgency

Medium (< 2 weeks)

Requested Change

Kindly remove these checks from branch protection rules, in the edx-notes-api repo

• build (py38, django32, mysql57)
• build (py38, django32, mysql80)

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

Django upgrade PR: openedx/edx-notes-api#349

[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.

[bmtcril]

@iamsobanjaved these checks have been removed for now. I'm going to keep this open as a reminder to follow up, but please let me know when I can enable the new checks.

[bmtcril]

@iamsobanjaved it looks like the 4.2 PR has merged, can I enable branch protections for build (py38, django42, mysql80)?

[iamsobanjaved]

Its not yet deployed on production and also hit an issue on stage. I will let you know once it reaches prod successfully

[sarina]

@iamsobanjaved is this issue ready to move forward?

[feanil]

@iamsobanjaved checking back in on this, are we clear to add django 4.2 tests as required for the notes api app?

[iamsobanjaved]

Yeah, we can mark Django 4.2 tests as required now.

[feanil]

This is all set.