This PR contains several improvements to our CI process which aim to make the CI workflow cleaner and to make it easier to configure required tests in code. ## Restructuring Before the change: <img width="1642" height="1161" alt="image" src="https://github.com/user-attachments/assets/4a0b9ddc-ed45-4e21-b4a7-50625ee0e350" /> After the change: <img width="1642" height="1161" alt="image" src="https://github.com/user-attachments/assets/6d3f37d2-0150-45f3-b8e1-5060180b555b" /> After the change, with some of the groups expanded: <img width="1642" height="1161" alt="image" src="https://github.com/user-attachments/assets/9727ebcb-d642-4989-882f-15b83e8e44f8" /> As can be observed in the screenshots, this restructuring makes it clearer what all the CI check is doing, when viewing the workflow summary in CI. The restructure also moves the Swift tests from a separate workflow file into the main `ci.yml` file. ## Functional changes This PR also adds a "Check required jobs" job, which passes only if all other jobs in `ci.yml` pass. We intend to make this check the only required status from `ci.yml` to make PRs mergeable. Making "Check required jobs" required will indirectly make the previously non-required checks required: Test Node, Test Swift, and the `-Funstable-mobile-app` tests. With this change, it is now also possible to configure the required checks in code. Also, this PR adds an optimization, so that any in progress jobs are cancelled if a new commit is pushed to the branch. Closes #2509 Closes #2558 Closes #2597

Uh oh!

Define required tests in code #2597

Description

Metadata

Metadata

Assignees

Labels

Fields

Projects

Milestone

Relationships

Development

Issue actions