[ALL-Feat] PR open 시 리뷰 요청 알림의 중복 전송을 방지 #47 #55
Merged
Conversation
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
skip 변수로 slack notify step의 실행 여부를 결정
action cache는 기존 캐시를 업데이트하거나 삭제하는 기능을 제공하지 않음. 따라서 첫 번째 캐싱된 리뷰어 목록이 계속 유지되는데, 다른 방안을 강구해야 할 듯함
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.
#️⃣ 연관된 이슈>
📝 작업 내용> 이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)
action cache를 사용해 특정 pr에 등록된 리뷰어 리스트를 캐싱하고, 이 리스트의 내용이 업데이트될 때만(diff가 생길 때만) 알림을 전송하도록 합니다.
🙏 여기는 꼭 봐주세요! > 리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
조금 전 알게 된 사실인데 action cache는 캐시를 업데이트하거나 삭제하는 기능을 제공하지 않는다고 하네요.. 일단 한 번에 여러 action workflow가 생성되면 가장 최근의 것만 실행하도록 해서 중복 문제를 임시로 해결해 놨고, 캐시 부분은 나중에 제대로 손봐야 할 것 같습니다.