You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
The text was updated successfully, but these errors were encountered:
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.
BilalQamar95
changed the title
[GH Request] <Remove "tests" as a required check in CI across multiple MFEs/JS libraries>
[GH Request] Remove tests as a required check in CI across multiple MFEs/JS libraries
Sep 16, 2024
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 thetests
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.The text was updated successfully, but these errors were encountered: