Skip to content

feat(ci): notify TechAPI to bump submodule on push to main#6

Merged
Seungpyo1007 merged 2 commits into
mainfrom
feat/notify-techapi
Jun 1, 2026
Merged

feat(ci): notify TechAPI to bump submodule on push to main#6
Seungpyo1007 merged 2 commits into
mainfrom
feat/notify-techapi

Conversation

@Seungpyo1007
Copy link
Copy Markdown
Member

목적

TechEngine main에 push될 때마다 TechAPI에 repository_dispatch(engine-updated)를 쏴서, TechAPI가 자기 쪽 TechEngine 서브모듈 포인터를 최신 커밋으로 자동 갱신하도록 함. 갱신 커밋은 TechAPI의 bump-engine.yml이 수행.

변경

  • 신규 notify-techapi.yml — push to main → peter-evans/repository-dispatch@v3engine-updated 디스패치 (payload: sha, ref). TECHAPI_TOKEN(Contents:write로 충분) 사용.
  • 정정 weekly-ingest.yml: TECHAPI_PR_TOKENTECHAPI_TOKEN. 실제 등록 시크릿과 이름이 안 맞아 ingest PR 단계가 조용히 skip되던 버그 수정.

비고

  • 무한루프 없음: TechAPI의 bump 커밋은 TechEngine에 push되지 않음.
  • 수신부(bump-engine.yml)가 TechAPI main에 있어야 실제 트리거됨(default 브랜치 한정). TechAPI 쪽에서 처리 중.
  • coverage-report.yml은 의도적으로 미변경 — TechAPI에 이슈를 생성하므로 Issues:write가 필요, 별도 결정.
  • 후속(별도 PR): refresh-data.yml 은퇴 검토.

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.
@Seungpyo1007 Seungpyo1007 merged commit a10726f into main Jun 1, 2026
2 checks passed
@Seungpyo1007 Seungpyo1007 deleted the feat/notify-techapi branch June 1, 2026 17:02
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