DEVX-56: Add --force flag to classify_changes and fix api_clients coverage #92

Merged
emil merged 0 commits from DEVX-56-classify-force-flag into master 2026-06-25 22:56:31 +00:00
Owner

Changes

  • Add --force flag to classify_changes that forces user-facing-changed=true in --github-output mode. Used by infra's workflow_dispatch with force-deploy input.
  • Add tests for --force flag (with/without --github-output, no tags)
  • Add tests for GiteaClient.list_prs (default and with params)
  • Add tests for VikunjaClient.list_comments and update_task_safe
  • Fix _strip_task_prefix regex to handle multi-segment task IDs (e.g. OBL-INFRA-192: not just GRM-51:)
  • Remove orphan sync_tasks.py and test_sync_tasks.py (untracked files from history rewrite, not referenced by any workflow)

Closes DEVX-56

## Changes - Add `--force` flag to `classify_changes` that forces `user-facing-changed=true` in `--github-output` mode. Used by infra's `workflow_dispatch` with `force-deploy` input. - Add tests for `--force` flag (with/without `--github-output`, no tags) - Add tests for `GiteaClient.list_prs` (default and with params) - Add tests for `VikunjaClient.list_comments` and `update_task_safe` - Fix `_strip_task_prefix` regex to handle multi-segment task IDs (e.g. `OBL-INFRA-192:` not just `GRM-51:`) - Remove orphan `sync_tasks.py` and `test_sync_tasks.py` (untracked files from history rewrite, not referenced by any workflow) Closes DEVX-56
emil added the ready-to-merge label 2026-06-25 22:51:53 +00:00
emil reviewed 2026-06-25 22:55:31 +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: 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 8450f33e88 into master 2026-06-25 22:56:31 +00:00
emil deleted branch DEVX-56-classify-force-flag 2026-06-25 22:56:32 +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#92