oblachno-oss/devx
Public Access
3
0
Fork 0
Code Issues 53 Pull Requests Actions Packages Projects Releases 10 Wiki Activity

DEVX-87: Add [skip ci] to release commits to prevent duplicate build-images runs #137

Closed
emil wants to merge 0 commits from DEVX-87-skip-ci-release-commits into master
pull from: DEVX-87-skip-ci-release-commits
merge into: :master
Conversation 1 Commits - Files Changed -
emil commented 2026-06-27 20:17:12 +00:00
Owner
Copy Link
Copy Source

Problem

Release commits (release: vX.Y.Z) change src/devx/__init__.py (version bump) which matches the src/devx/** path filter in build-images.yml. This causes build-images to run twice for each PR merge: once for the merge commit and once for the release commit.

Fix

  1. Add [skip ci] to the release commit message in devx.ci.release.commit_release_changes() — prevents CI workflows from triggering on release commits
  2. Add concurrency group to build-images.yml — second layer of defense against duplicate runs

Closes DEVX-87

## Problem Release commits (`release: vX.Y.Z`) change `src/devx/__init__.py` (version bump) which matches the `src/devx/**` path filter in build-images.yml. This causes build-images to run twice for each PR merge: once for the merge commit and once for the release commit. ## Fix 1. Add `[skip ci]` to the release commit message in `devx.ci.release.commit_release_changes()` — prevents CI workflows from triggering on release commits 2. Add `concurrency` group to `build-images.yml` — second layer of defense against duplicate runs Closes DEVX-87
emil force-pushed DEVX-87-skip-ci-release-commits from c046a0b334 to 477010f89e 2026-06-27 20:36:55 +00:00 Compare
emil reviewed 2026-06-27 20:42:47 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

All 13 REVIEW_CHECKLIST.md categories verified. Architecture: single-line change to release commit message format plus concurrency group in workflow. Security: no secrets exposed, [skip ci] is a standard convention. Tests: existing test updated to match new format. Docs: AGENTS.md already documents [skip ci] convention. Backwards compat: regex patterns use prefix match, [skip ci] suffix does not break them.

All 13 REVIEW_CHECKLIST.md categories verified. Architecture: single-line change to release commit message format plus concurrency group in workflow. Security: no secrets exposed, [skip ci] is a standard convention. Tests: existing test updated to match new format. Docs: AGENTS.md already documents [skip ci] convention. Backwards compat: regex patterns use prefix match, [skip ci] suffix does not break them.
emil added the ready-to-merge label 2026-06-27 20:42:51 +00:00
emil force-pushed DEVX-87-skip-ci-release-commits from 084944701a to fd7eb73f9a 2026-06-27 20:51:10 +00:00 Compare
emil closed this pull request 2026-06-27 20:59:30 +00:00
emil reopened this pull request 2026-06-27 20:59:34 +00:00
emil closed this pull request 2026-06-27 21:03:43 +00:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something is not working
 ci-improvement
CI workflow improvements
 doc-improvement
Documentation improvements
 feedback
Issues from subagent or agent feedback
 ready-to-merge
tooling
Tool-related feedback or improvements
 workflow-improvement
Workflow alignment or pattern improvements
No labels ready-to-merge
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
emil (emil User) emo kireto
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: oblachno-oss/devx#137