Comprehensive audit of all devx CI tools, converting warnings to errors and adding missing input validation.
Changes by tool:
check_translations: Missing languages and dead keys are now errors (were warnings). Removed --strict flag. Fixed translations.json (removed 19 dead keys, added 11 missing keys, added 408 missing language entries).
post_merge: Non-infrastructure commits without task ID now raise (was warn+skip). Vikunja API errors now raise (was warn+return).
auto_merge: Missing VIKUNJA_TOKEN now raises (was silently skip). Added PR number and repo format validation.
release: Empty changelog now raises (was warn+continue). Added version format validation.
discover_runners: Silent except: pass replaced with logged warnings.
api_clients: Bare except Exception replaced with specific exception types.
notify_failure: contextlib.suppress replaced with explicit try/except + logging.
molecule_ci_guard: Added pair format validation.
distribute_molecule: Added runner index bounds check.
All 821 tests pass with 100% coverage.
Closes DEVX-7
## Summary
Comprehensive audit of all devx CI tools, converting warnings to errors and adding missing input validation.
### Changes by tool:
- **check_translations**: Missing languages and dead keys are now errors (were warnings). Removed `--strict` flag. Fixed translations.json (removed 19 dead keys, added 11 missing keys, added 408 missing language entries).
- **post_merge**: Non-infrastructure commits without task ID now raise (was warn+skip). Vikunja API errors now raise (was warn+return).
- **auto_merge**: Missing VIKUNJA_TOKEN now raises (was silently skip). Added PR number and repo format validation.
- **release**: Empty changelog now raises (was warn+continue). Added version format validation.
- **sync_wiki**: Missing/empty mapped files now raise (was warn+skip). Added mapping.json structure validation.
- **discover_runners**: Silent `except: pass` replaced with logged warnings.
- **api_clients**: Bare `except Exception` replaced with specific exception types.
- **notify_failure**: `contextlib.suppress` replaced with explicit try/except + logging.
- **molecule_ci_guard**: Added pair format validation.
- **distribute_molecule**: Added runner index bounds check.
All 821 tests pass with 100% coverage.
Closes DEVX-7
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 007e8dce9f into master2026-06-22 20:54:26 +00:00
emil
deleted branch DEVX-7-warnings-to-errors2026-06-22 20:54:26 +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
Summary
Comprehensive audit of all devx CI tools, converting warnings to errors and adding missing input validation.
Changes by tool:
--strictflag. Fixed translations.json (removed 19 dead keys, added 11 missing keys, added 408 missing language entries).except: passreplaced with logged warnings.except Exceptionreplaced with specific exception types.contextlib.suppressreplaced with explicit try/except + logging.All 821 tests pass with 100% coverage.
Closes DEVX-7
Automated PR Review
No issues found by automated checks.
Auto-merge: If all CI checks pass, this PR will be merged automatically.