oblachno-oss/grm
3
0
Fork 0
Code Issues 21 Pull Requests Actions Packages Projects Releases 15 Wiki Activity

GRM-76: Retrospective fixes for CI/CD friction #112

Merged
emil merged 0 commits from GRM-76-retrospective-fixes into master 2026-06-24 16:47:02 +00:00
Conversation 4 Commits - Files Changed -
emil commented 2026-06-24 16:29:20 +00:00
Owner
Copy Link
Copy Source

Summary

Retrospective fixes for CI/CD friction:

  • Update devx from 0.6.0 to 0.9.8 — picks up all auto_merge, configure_repo, and config fixes
  • auto-merge: Add molecule-tests to needs and use always() so auto-merge evaluates even when molecule-tests is skipped
  • configure-repo: Add DEVX_REPO_OWNER=oblachno-oss explicitly (no longer defaults in devx)
  • Docs: Fix review_pr → pr_review references (the module is pr_review.py)

Closes GRM-76

## Summary Retrospective fixes for CI/CD friction: - **Update devx from 0.6.0 to 0.9.8** — picks up all auto_merge, configure_repo, and config fixes - **auto-merge**: Add `molecule-tests` to `needs` and use `always()` so auto-merge evaluates even when molecule-tests is skipped - **configure-repo**: Add `DEVX_REPO_OWNER=oblachno-oss` explicitly (no longer defaults in devx) - **Docs**: Fix `review_pr` → `pr_review` references (the module is `pr_review.py`) Closes GRM-76
emil reviewed 2026-06-24 16:29:32 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

Retrospective fixes. All 13 categories verified. Architecture: devx version bump picks up fixes, auto-merge now gates on molecule-tests. Security: no hardcoded secrets. Tests: pre-commit hooks pass. Docs: review_pr references corrected to pr_review.

Retrospective fixes. All 13 categories verified. Architecture: devx version bump picks up fixes, auto-merge now gates on molecule-tests. Security: no hardcoded secrets. Tests: pre-commit hooks pass. Docs: review_pr references corrected to pr_review.
emil reviewed 2026-06-24 16:29:32 +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 added the ready-to-merge label 2026-06-24 16:29:41 +00:00
emil reviewed 2026-06-24 16:41:02 +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-24 16:45: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 merged commit f7e53e4f93 into master 2026-06-24 16:47:02 +00:00
emil deleted branch GRM-76-retrospective-fixes 2026-06-24 16:47:03 +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
Auto-merge PR when all CI checks pass
 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/grm#112