DEVX-55: Revert squash-merge format to use colon after task ID #88

Merged
emil merged 1 commits from DEVX-55-revert-squash-format into master 2026-06-25 22:09:47 +00:00
Owner

Problem

DEVX-54 incorrectly changed the squash-merge format from TASK-N: <conventional commit> to TASK-N <conventional commit> (removing the colon). The colon format is the established convention across all three repos (devx, grm, infra) and is already used by the cliff.toml preprocessors.

Fix

Revert auto_merge.py to produce {task_id}: {conv_msg} (with colon). Update AGENTS.md to document the colon format.

Closes DEVX-55

## Problem DEVX-54 incorrectly changed the squash-merge format from `TASK-N: <conventional commit>` to `TASK-N <conventional commit>` (removing the colon). The colon format is the established convention across all three repos (devx, grm, infra) and is already used by the cliff.toml preprocessors. ## Fix Revert auto_merge.py to produce `{task_id}: {conv_msg}` (with colon). Update AGENTS.md to document the colon format. Closes DEVX-55
emil added the ready-to-merge label 2026-06-25 22:02:39 +00:00
emil reviewed 2026-06-25 22:03:20 +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: OK
  • 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: OK - 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 added 1 commit 2026-06-25 22:07:39 +00:00
fix: revert squash-merge format to use colon after task ID
CI / detect-changes (pull_request) Successful in 56s
CI / pr-review (pull_request) Successful in 50s
CI / quality (pull_request) Successful in 1m54s
CI / auto-merge (pull_request) Successful in 10s
CI / release-dry-run (pull_request) Successful in 55s
c7375fbecd
DEVX-54 incorrectly changed the squash-merge format from
'TASK-N: <conventional commit>' to 'TASK-N <conventional commit>'.
Revert to the colon format which is the established convention
across devx, grm, and infra. Update AGENTS.md docs to match.
emil force-pushed DEVX-55-revert-squash-format from 14fc73abc6 to c7375fbecd 2026-06-25 22:07:39 +00:00 Compare
emil reviewed 2026-06-25 22:09:29 +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: OK
  • 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: OK - 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 95384c26e1 into master 2026-06-25 22:09:47 +00:00
emil deleted branch DEVX-55-revert-squash-format 2026-06-25 22:09:47 +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#88