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

DEVX-65: Consolidate publish.yml into post-merge.yml #105

Merged
emil merged 1 commits from DEVX-65-consolidate-publish-into-post-merge into master 2026-06-26 19:34:47 +00:00
Conversation 2 Commits 1 Files Changed 6
+96 -55
emil commented 2026-06-26 19:32:51 +00:00
Owner
Copy Link
Copy Source
No description provided.
emil added 1 commit 2026-06-26 19:32:52 +00:00
refactor: consolidate publish.yml into post-merge.yml
CI / detect-changes (pull_request) Successful in 38s
Details
CI / pr-review (pull_request) Successful in 47s
Details
CI / quality (pull_request) Successful in 1m14s
Details
CI / release-dry-run (pull_request) Successful in 31s
Details
CI / auto-merge (pull_request) Successful in 40s
Details
340ac80a3e 
Merge the separate publish workflow into a publish job in post-merge.yml
that depends on release. The release job now writes the tag to
GITHUB_OUTPUT, and the publish job reads it via needs.release.outputs.tag.

This eliminates the tag-push → publish.yml indirection, making the full
release pipeline visible in a single workflow run.

Changes:
- Add _write_github_output() to release.py — writes tag to GITHUB_OUTPUT
- Add `tag` output to release job in post-merge.yml
- Add publish job (needs: release) to post-merge.yml
- Delete publish.yml
- Update AGENTS.md docs
- Add tests for GITHUB_OUTPUT writing

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
emil reviewed 2026-06-26 19:33:40 +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 7c1ecd6ff9 into master 2026-06-26 19:34:47 +00:00
emil deleted branch DEVX-65-consolidate-publish-into-post-merge 2026-06-26 19:34:47 +00:00
emil reviewed 2026-06-26 19:36:51 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

All 13 REVIEW_CHECKLIST.md categories verified. Architecture: publish job added as downstream of release, GITHUB_OUTPUT wiring added to release.py. Security: no new secrets or exposures. Tests: 1293 tests, 100% coverage, new tests for _write_github_output. Docs: AGENTS.md updated.

All 13 REVIEW_CHECKLIST.md categories verified. Architecture: publish job added as downstream of release, GITHUB_OUTPUT wiring added to release.py. Security: no new secrets or exposures. Tests: 1293 tests, 100% coverage, new tests for _write_github_output. Docs: AGENTS.md updated.
emil added the ready-to-merge label 2026-06-26 19:36:52 +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) kireto
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: oblachno-oss/devx#105