feat(ci): notify TechAPI to bump submodule on push to main#6
Merged
Conversation
Add notify-techapi.yml: on every push to main, send a repository_dispatch (engine-updated) to Seungpyo1007/TechAPI so its bump-engine.yml advances the TechEngine submodule pointer to the new commit. Uses the existing TECHAPI_TOKEN (Contents:write is sufficient for the dispatches API). Also rename TECHAPI_PR_TOKEN -> TECHAPI_TOKEN in weekly-ingest.yml to match the actual secret; the mismatch was silently skipping the ingest PR step. coverage-report.yml is left untouched (its issue-posting needs Issues:write, decided separately).
Finish the token-name cleanup in coverage-report.yml (TECHAPI_PR_TOKEN -> TECHAPI_TOKEN) and post the sticky coverage issue to BOTH repos: TechEngine via GITHUB_TOKEN (works today) and TechAPI via TECHAPI_TOKEN. The TechAPI post is best-effort and warns instead of failing when the PAT lacks Issues:write, so the weekly run never breaks.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
목적
TechEngine main에 push될 때마다 TechAPI에
repository_dispatch(engine-updated)를 쏴서, TechAPI가 자기 쪽 TechEngine 서브모듈 포인터를 최신 커밋으로 자동 갱신하도록 함. 갱신 커밋은 TechAPI의bump-engine.yml이 수행.변경
notify-techapi.yml— push to main →peter-evans/repository-dispatch@v3로engine-updated디스패치 (payload: sha, ref).TECHAPI_TOKEN(Contents:write로 충분) 사용.weekly-ingest.yml:TECHAPI_PR_TOKEN→TECHAPI_TOKEN. 실제 등록 시크릿과 이름이 안 맞아 ingest PR 단계가 조용히 skip되던 버그 수정.비고
bump-engine.yml)가 TechAPI main에 있어야 실제 트리거됨(default 브랜치 한정). TechAPI 쪽에서 처리 중.coverage-report.yml은 의도적으로 미변경 — TechAPI에 이슈를 생성하므로 Issues:write가 필요, 별도 결정.refresh-data.yml은퇴 검토.