Skip to content

ci: add TechEngineBot final-check comment for important PRs - #35

Closed
Seungpyo1007 wants to merge 2 commits into
developfrom
ci/final-check
Closed

Seungpyo1007 wants to merge 2 commits into
developfrom
ci/final-check

Conversation

@Seungpyo1007

Copy link
Copy Markdown
Member

What & why

TechEngineBot acts as the final reviewer on important PRs: it leaves one comment summarising the validator/test results and the ES records the PR adds, modifies, or removes. The comment is edited in place on every push, never duplicated.

  • Triggers: release PRs into main, and any PR labeled final-check (drafts are skipped).
  • Posts with the TECHENGINEBOT_TOKEN secret only — never a personal token. Without the secret the job still runs and skips the comment.
  • The job fails when validation or tests fail.

Setup

Add a TechEngineBot token (public_repo or fine-grained "Pull requests: write" on this repo) as the repo secret TECHENGINEBOT_TOKEN.

Checklist

  • python -m app.validate passes locally
  • No data changes

Release PRs into main and PRs labeled final-check get a single comment
from TechEngineBot with validator/test results and the records touched.
The comment is edited in place on each push. Posting needs the
TECHENGINEBOT_TOKEN secret; without it the job skips the comment.
@Seungpyo1007 Seungpyo1007 added ci CI and workflow changes final-check labels Oct 29, 2026
Without pipefail a failed gh api call in the lookup pipeline left its
error body in $id, and the step then sent it to PATCH as a comment id.
@Seungpyo1007
Seungpyo1007 deleted the ci/final-check branch October 31, 2026 14:18
@TechEngineBot TechEngineBot moved this to Done in TechAPI-Project Nov 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci CI and workflow changes

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant