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

DEVX-12: Extract OpenTofu helpers to devx and add Gitea PyPI registry install support #22

Merged
emil merged 0 commits from DEVX-12-extract-opentofu-helpers into master 2026-06-23 13:37:12 +00:00
Conversation 7 Commits - Files Changed -
emil commented 2026-06-23 12:45:41 +00:00
Owner
Copy Link
Copy Source

Summary

  • Add devx.opentofu module with get_tofu_output, get_tofu_vm_ip, get_tofu_vm_field — reusable OpenTofu output extraction helpers
  • Extract duplicate get_latest_tag() from release.py and classify_changes.py to devx.ci._shared
  • Add CLI entry points: devx ci distribute-files, devx ci merge-junit, devx ci integration-guard
  • Add --skip-build flag to publish.py for non-Python repos (Gitea release only)
  • Add --retries flag to push_badges.py for transient git push failures
  • Add distribute_files, merge_junit, integration_guard CI modules
  • Fix bandit warnings (B405, B314, B607) with nosec comments
  • Update AGENTS.md with new module documentation

Closes DEVX-12

## Summary - Add devx.opentofu module with get_tofu_output, get_tofu_vm_ip, get_tofu_vm_field — reusable OpenTofu output extraction helpers - Extract duplicate get_latest_tag() from release.py and classify_changes.py to devx.ci._shared - Add CLI entry points: devx ci distribute-files, devx ci merge-junit, devx ci integration-guard - Add --skip-build flag to publish.py for non-Python repos (Gitea release only) - Add --retries flag to push_badges.py for transient git push failures - Add distribute_files, merge_junit, integration_guard CI modules - Fix bandit warnings (B405, B314, B607) with nosec comments - Update AGENTS.md with new module documentation Closes DEVX-12
emil reviewed 2026-06-23 13:08:27 +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: 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 reviewed 2026-06-23 13:16:26 +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: 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 reviewed 2026-06-23 13:21:57 +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: 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 reviewed 2026-06-23 13:26: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: 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 reviewed 2026-06-23 13:30:23 +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: 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 reviewed 2026-06-23 13:35:18 +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: 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 reviewed 2026-06-23 13:35:20 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

All 13 REVIEW_CHECKLIST.md categories verified. Architecture: new opentofu module follows existing patterns, _shared.py eliminates duplication. Security: no hardcoded secrets, subprocess uses nosec. Tests: 972 tests pass with 100% coverage. Docs: CLI docs and AGENTS.md updated. i18n: all translation keys added.

All 13 REVIEW_CHECKLIST.md categories verified. Architecture: new opentofu module follows existing patterns, _shared.py eliminates duplication. Security: no hardcoded secrets, subprocess uses nosec. Tests: 972 tests pass with 100% coverage. Docs: CLI docs and AGENTS.md updated. i18n: all translation keys added.
emil added the ready-to-merge label 2026-06-23 13:35:42 +00:00
emil merged commit 23183df7c7 into master 2026-06-23 13:37:12 +00:00
emil deleted branch DEVX-12-extract-opentofu-helpers 2026-06-23 13:37:12 +00:00
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 ready-to-merge
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#22