search

openedx / public-engineering

General public issue repository for the Open edX engineering community
4 stars 2 forks source link

[GH Request] Remove `tests` as a required check in CI across multiple MFEs/JS libraries #282

Closed BilalQamar95 closed 1 week ago

BilalQamar95 commented 1 month ago

Repository

axim-engineering

Urgency

High (< 3 days)

Requested Change

Remove the tests status check as a required check in CI. This will prevent pipeline hang-ups and allow non-blocking test runs for Node v20 as part of the transitional phase where both Node v18 and Node v20 are in the CI matrix.

Reasoning

As part of the Node v20 upgrade process, Node v20 was added to the CI test matrix alongside Node v18. This has caused the tests status check to remain in a pending state, leading to pipeline hang-ups. In MFEs and JS libraries where the tests status check is marked as required, it should be removed to allow for test runs during this transitional phase.

Removing the tests check will prevent the CI pipeline from hanging while both Node v18 and Node v20 are tested, allowing the Node v20 upgrade process to move forward smoothly.

github-actions[bot] commented 1 month 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.