[PL-408] CI 커버리지 코멘트 단계에 워크플로 권한 선언 추가 - #44
Merged
Conversation
☂️ Code Coverage
Overall Coverage
New FilesNo new covered files... Modified FilesNo covered modified files...
|
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.
Description 🔍
배경
조직 기본 워크플로 토큰 권한을 2026-08-20 에
write에서read로 내렸습니다. 공급망 위험을 줄이기 위한 조직 차원의 변경입니다.이 저장소는 워크플로에
permissions를 선언하지 않아 조직 기본값을 그대로 받습니다. 그래서 커버리지 리포트 단계가 PR 에 코멘트를 달 때Resource not accessible by integration으로 실패합니다.같은 원인으로
fep-kftc의 CI 가 실패했고, 해당 저장소는 같은 방식으로 이미 수정했습니다.central-record에서도 동일한 수정으로 CI 통과를 확인했습니다.변경 사항
커버리지 리포트를 수행하는 잡이 PR 코멘트를 달 수 있도록 필요한 권한만 선언합니다.
나머지 잡은 쓰기가 필요 없으므로 조직 기본값인
read를 그대로 씁니다.구현 내용
.github/workflows/ci.yml: 커버리지 리포트 잡에permissions블록을 추가합니다.contents: read를 함께 적는 이유는,permissions를 선언하면 적지 않은 스코프가none이 되어 체크아웃이 막히기 때문입니다.참고 자료