Commit d93f43e
committed
Add breaking change detection CI check
Adds a new 'major-change-check' job to the PR workflow that detects
potential breaking changes by comparing the branch against main:
1. Changesets requesting a major version bump
2. OCLIF manifest: removed commands, flags, or flag env vars
3. Zod schemas: removed fields in app/extension config specifications
When breaking changes are detected:
- A sticky PR comment is posted pinging @Shopify/dev_experience to
coordinate inclusion in the next major release
- The check fails (red) until a dev_experience team member approves
the workflow run via the 'breaking-change-approval' environment gate
- Comment links to #help-dev-platform for coordinating major releases
When no breaking changes are found, any stale sticky comment from a
previous run is deleted.1 parent 2f9bc85 commit d93f43e
2 files changed
Lines changed: 425 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
287 | 287 | | |
288 | 288 | | |
289 | 289 | | |
| 290 | + | |
| 291 | + | |
| 292 | + | |
| 293 | + | |
| 294 | + | |
| 295 | + | |
| 296 | + | |
| 297 | + | |
| 298 | + | |
| 299 | + | |
| 300 | + | |
| 301 | + | |
| 302 | + | |
| 303 | + | |
| 304 | + | |
| 305 | + | |
| 306 | + | |
| 307 | + | |
| 308 | + | |
| 309 | + | |
| 310 | + | |
| 311 | + | |
| 312 | + | |
| 313 | + | |
| 314 | + | |
| 315 | + | |
| 316 | + | |
| 317 | + | |
| 318 | + | |
| 319 | + | |
| 320 | + | |
| 321 | + | |
| 322 | + | |
| 323 | + | |
| 324 | + | |
| 325 | + | |
| 326 | + | |
| 327 | + | |
| 328 | + | |
| 329 | + | |
| 330 | + | |
| 331 | + | |
| 332 | + | |
| 333 | + | |
| 334 | + | |
| 335 | + | |
| 336 | + | |
| 337 | + | |
| 338 | + | |
| 339 | + | |
0 commit comments