oblachno-oss/devx
Public Access
2
0
Fork 0
Code Issues 30 Pull Requests Actions Packages Projects Releases 2 Wiki Activity

DEVX-31: Fix release branch check for heads/ prefix #47

Closed
emil wants to merge 1 commits from DEVX-31-fix-branch-check into master
pull from: DEVX-31-fix-branch-check
merge into: :master
Conversation 5 Commits 1 Files Changed 2
+3 -1
emil commented 2026-06-24 10:38:40 +00:00
Owner
Copy Link
Copy Source

Some git versions return 'heads/master' instead of 'master' from git rev-parse --abbrev-ref HEAD. Strip the heads/ prefix before comparing.

Closes DEVX-31

Some git versions return 'heads/master' instead of 'master' from git rev-parse --abbrev-ref HEAD. Strip the heads/ prefix before comparing. Closes DEVX-31
emil reviewed 2026-06-24 10:39:17 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

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: WARNING — source files changed but no test files updated
  • 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: WARNING — source files changed but no test files updated - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil reviewed 2026-06-24 10:43:15 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

Single-line fix to strip heads/ prefix. Tests pass. All categories verified.

Single-line fix to strip heads/ prefix. Tests pass. All categories verified.
emil added the ready-to-merge label 2026-06-24 10:43:34 +00:00
emil force-pushed DEVX-31-fix-branch-check from d940a6aebc to 39102590a6 2026-06-24 10:45:49 +00:00 Compare
emil reviewed 2026-06-24 10:46:30 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

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: WARNING — source files changed but no test files updated
  • 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: WARNING — source files changed but no test files updated - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil force-pushed DEVX-31-fix-branch-check from 39102590a6 to 3c9c614813 2026-06-24 10:46:59 +00:00 Compare
emil reviewed 2026-06-24 10:47:36 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

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: WARNING — source files changed but no test files updated
  • 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: WARNING — source files changed but no test files updated - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil added 1 commit 2026-06-24 10:50:00 +00:00
fix: strip heads/ prefix from branch name in release check
CI / detect-changes (pull_request) Successful in 28s
Details
CI / pr-review (pull_request) Successful in 35s
Details
CI / quality (pull_request) Successful in 1m9s
Details
CI / auto-merge (pull_request) Failing after 13s
Details
CI / release-dry-run (pull_request) Successful in 27s
Details
402b8b0578
emil force-pushed DEVX-31-fix-branch-check from 3c9c614813 to 402b8b0578 2026-06-24 10:50:00 +00:00 Compare
emil reviewed 2026-06-24 10:50:37 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

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: WARNING — source files changed but no test files updated
  • 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: WARNING — source files changed but no test files updated - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil closed this pull request 2026-06-24 10:50:47 +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
emil
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) kireto
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: oblachno-oss/devx#47