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

DEVX-31: Use branch name as sole task ID source #52

Closed
emil wants to merge 0 commits from DEVX-31-taskid-fix into master
pull from: DEVX-31-taskid-fix
merge into: :master
Conversation 2 Commits - Files Changed -
emil commented 2026-06-24 11:03:29 +00:00
Owner
Copy Link
Copy Source

The .taskid file caused merge conflicts on every PR. Branch names already contain the task ID, so .taskid is redundant.

  • read_taskid() now only extracts from branch name
  • Fix release.py: strip heads/ prefix from git rev-parse output
  • Update translations and tests

Closes DEVX-31

The .taskid file caused merge conflicts on every PR. Branch names already contain the task ID, so .taskid is redundant. - read_taskid() now only extracts from branch name - Fix release.py: strip heads/ prefix from git rev-parse output - Update translations and tests Closes DEVX-31
emil reviewed 2026-06-24 11:04:11 +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: 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 force-pushed DEVX-31-taskid-fix from 6a43e63ce6 to 55a12fb9ad 2026-06-24 11:04:28 +00:00 Compare
emil reviewed 2026-06-24 11:05:04 +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: 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 closed this pull request 2026-06-24 11:05:51 +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
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#52