The release workflow crashed with ValueError when non-version tags (e.g., 'master') existed in the repo. verify_tag_consistency() tried to parse all tags as version numbers.
Fix: filter to only vX.Y.Z tags before processing.
Closes DEVX-32
The release workflow crashed with ValueError when non-version tags (e.g., 'master') existed in the repo. verify_tag_consistency() tried to parse all tags as version numbers.
Fix: filter to only vX.Y.Z tags before processing.
Closes DEVX-32
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 131c04c9d0 into master2026-06-24 11:09:48 +00:00
emil
deleted branch DEVX-32-fix-tag-crash2026-06-24 11:09:48 +00:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
emil
The release workflow crashed with ValueError when non-version tags (e.g., 'master') existed in the repo. verify_tag_consistency() tried to parse all tags as version numbers.
Fix: filter to only vX.Y.Z tags before processing.
Closes DEVX-32
Fix filters non-version tags in verify_tag_consistency and verify_alignment. Tests cover the new case. All categories verified.
Automated PR Review
No issues found by automated checks.
Auto-merge: If all CI checks pass, this PR will be merged automatically.