Skip to content

Switch PR review to Hermes - #59

Merged
midagedev merged 6 commits into
mainfrom
codex/hermes-pr-review
May 31, 2026
Merged

Switch PR review to Hermes#59
midagedev merged 6 commits into
mainfrom
codex/hermes-pr-review

Conversation

@midagedev

Copy link
Copy Markdown
Owner

Summary\n- switch the AI PR review caller from the retired Z.ai workflow to the Hermes reusable workflow\n- target the VPS self-hosted runner registered for billtap with labels vps-hermes/hermes/billtap\n- keep the existing repository-specific review prompt during migration\n\n## Verification\n- ruby YAML parse for .github/workflows/zai-review.yml\n- confirmed the billtap self-hosted runner is online in GitHub Actions

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

APPROVE
워크플로우 전환 범위가 명확하고, 리뷰 프롬프트 경로와 self-hosted 러너 라벨도 의도대로 유지/갱신되었습니다.
Reviewed by Hermes via OpenAI Codex

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

APPROVE
Hermes reusable workflow로의 전환과 self-hosted runner 라벨 지정이 의도대로 보입니다. 블로킹 이슈는 보이지 않습니다.
Reviewed by Hermes via OpenAI Codex

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

APPROVE
.github/workflows/zai-review.yml의 리뷰 호출 대상만 Hermes reusable workflow로 전환되어 있고, 빌링/웹훅/보안 회귀를 유발할 변경은 보이지 않습니다.
Reviewed by Hermes via OpenAI Codex

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

APPROVE
blocking 이슈는 보이지 않습니다. Hermes reusable workflow로의 전환, self-hosted runner 라벨 지정, 기존 리뷰 프롬프트 유지 모두 의도에 맞게 보입니다.
Reviewed by Hermes via OpenAI Codex

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

APPROVE
.github/workflows/zai-review.yml에서 호출 워크플로우와 러너 라벨만 Hermes용으로 전환했고, PR 리뷰 트리거/가드 조건은 유지되어 보입니다. 별도 차단 이슈는 확인되지 않았습니다.
Reviewed by Hermes via OpenAI Codex

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

APPROVE
변경은 Hermes 워크플로우로의 전환과 러너 라벨 조정에 국한되어 있으며, 차단 이슈는 확인되지 않았습니다.

Reviewed by Hermes via OpenAI Codex

@midagedev
midagedev merged commit a669bb2 into main May 31, 2026
1 of 2 checks passed
@midagedev
midagedev deleted the codex/hermes-pr-review branch May 31, 2026 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant