🚀 Cancel stale CI pipelines - #211
Conversation
There was a problem hiding this comment.
PR Review — Score: 4.9 / 5
This is a focused CI hygiene change: default jobs are interruptible, superseded branch pipelines auto-cancel, and main/tags plus the manual build-container-image job are explicitly protected. The workflow rules are correctly scoped and align with the established dd-sdk-android pattern. I would approve.
Why 4.9: Minimal, well-commented diff with the right exception for long-running manual Docker builds, and the author validated YAML structure locally. No SDK or customer-facing impact.
Why not 5: CI-only configuration changes inherently lack in-repo regression tests beyond static parsing; operational correctness depends on GitLab runtime behavior. Several dd-gitlab/* jobs were still running when this review was posted (not a merge blocker).
Sent by Cursor Automation: electron-sdk reviews
cd7e4c8 to
6b110e7
Compare


Motivation
Stale branch pipelines keep consuming CI capacity after newer commits supersede them.
Changes
interruptible: true.mainbranches when a new commit is pushed.mainand tags.This follows the configuration already used in
dd-sdk-androidand DataDog/shopist-android#257.Test instructions
yarn prettier --check .gitlab-ci.yml.gitlab-ci.ymlwith Ruby YAML.Checklist
.gitlab-ci.yml.