Conversation
pull_request 체크아웃의 HEAD 는 base 와 미리 합쳐본 머지 커밋이다. 그걸 끝점으로 쓰면 PR 이 열려 있는 동안 main 에 들어온 테스트까지 이 PR 이 고친 것으로 잡힌다. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WU6a447YMHF72RTcCDW2nr
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
1 participant
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.
Closes #36
무엇을 바꿨나
테스트 명세 잠금단계가 비교하는 끝점을HEAD에서github.event.pull_request.head.sha로 바꿨다.pull_request체크아웃의HEAD는 브랜치 끝이 아니라 base 와 미리 합쳐본 머지 커밋이다.그래서
FIRST..HEAD에는 PR 이 열려 있는 동안main에 들어온 커밋이 함께 들어오고,그 커밋이 테스트를 건드렸으면 이 PR 이 고친 것으로 잡혔다.
npm test와 e2e 잡은 그대로 머지 커밋에서 돈다. 잠금 검사만 브랜치 커밋을 본다.확인
npm test137개 통과.바뀐 명령을 로컬에서 두 경우에 돌렸다.
main에 테스트가 머지됨) → 결과 없음이 PR 자체의 CI 가 통과하는 것이 첫 번째 확인이다.
미확인
머지 후 PR #35 의 CI. 워크플로는 머지 커밋 기준으로 읽히므로, 이 PR 이
main에들어간 뒤 #35 를 다시 돌려야 초록이 된다.
🤖 Generated with Claude Code
https://claude.ai/code/session_01WU6a447YMHF72RTcCDW2nr