[Task]: Trigger cypress CI workflows using commit message - Githubissues

appsmithorg / appsmith

Platform to build admin panels, internal tools, and dashboards. Integrates with 25+ databases and any API.

https://www.appsmith.com

Apache License 2.0

34.63k stars 3.74k forks source link

[Task]: Trigger cypress CI workflows using commit message #35663

Open rishabhrathod01 opened 3 months ago

rishabhrathod01 commented 3 months ago

We have made changes in CI to trigger cypress using commit, current logic is

Add the ok-to-test label manually on PR
After this label is added each commit will lead to cypress workflow to run.
It makes sure that it cancels previous workflow when next commit is made

Future Solution

[x] Trigger the cypress test with the commit itself but with a particular message in the commit like ‘trigger-cypress’. If the commit message has ‘trigger-cypress’ it runs the cypress workflow.
[x] Allows tags to be mentioned in the commit message itself.
[ ] Make sure the PR checks passes
[ ] Make sure the PR description is not updated
[ ] Comment for failure cases

Why work on this?

Fix the external contribution unable to run CI checks using PR from forked repo.
Balance between best DX and resource consumption, current implementation needs manual addition of label to trigger first cypress and also manual update of description required to mention tags for spec that need to run.

rishabhrathod01 commented 1 month ago

The initial POC was done here to trigger the workflow using commit messages. https://github.com/appsmithorg/appsmith/pull/36434#issuecomment-2369064696

Refer to it for the next steps.