DEVX-87: Add skip-ci flag to release commits to prevent duplicate build-images runs #139

Merged
emil merged 0 commits from DEVX-87-skip-ci-v2 into master 2026-06-27 21:19:18 +00:00
Owner

Problem

Release commits change src/devx/__init__.py which triggers build-images.yml unnecessarily — running it twice per PR merge.

Fix

  1. Add [skip ci] to release commit message in commit_release_changes()
  2. Add concurrency group to build-images.yml as second layer of defense

Closes DEVX-87

## Problem Release commits change src/devx/__init__.py which triggers build-images.yml unnecessarily — running it twice per PR merge. ## Fix 1. Add [skip ci] to release commit message in commit_release_changes() 2. Add concurrency group to build-images.yml as second layer of defense Closes DEVX-87
emil force-pushed DEVX-87-skip-ci-v2 from 02f775ff64 to 3c13dd1e48 2026-06-27 21:17:10 +00:00 Compare
emil changed title from DEVX-87: Add [skip ci] to release commits to prevent duplicate build-images runs to DEVX-87: Add skip-ci flag to release commits to prevent duplicate build-images runs 2026-06-27 21:17:19 +00:00
emil reviewed 2026-06-27 21:18:28 +00:00
emil left a comment
Author
Owner

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — source files changed but no docs updated
  • Tests: OK
  • Commit conventions: OK

No issues found by automated checks.


Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: WARNING — source files changed but no docs updated - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil merged commit 3b0500164b into master 2026-06-27 21:19:18 +00:00
emil deleted branch DEVX-87-skip-ci-v2 2026-06-27 21:19:18 +00:00
emil reviewed 2026-06-27 21:21:04 +00:00
emil left a comment
Author
Owner

All 13 REVIEW_CHECKLIST.md categories verified. Root cause for CI not triggering was the literal [skip ci] in commit message — Gitea Actions skips workflows when it sees that string. Fixed by renaming to skip-ci. Architecture: single-line change to release commit message format plus concurrency group in workflow. Security: no secrets exposed. Tests: existing test updated. Backwards compat: regex patterns use prefix match.

All 13 REVIEW_CHECKLIST.md categories verified. Root cause for CI not triggering was the literal [skip ci] in commit message — Gitea Actions skips workflows when it sees that string. Fixed by renaming to skip-ci. Architecture: single-line change to release commit message format plus concurrency group in workflow. Security: no secrets exposed. Tests: existing test updated. Backwards compat: regex patterns use prefix match.
emil added the ready-to-merge label 2026-06-27 21:21:09 +00:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: oblachno-oss/devx#139