Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat: sync-upstream.yml #5

Open
wants to merge 1 commit into
base: master-ko
Choose a base branch
from

Conversation

minchodang
Copy link
Member

  • react-hook-form upstream upstream master와 react-ko-form origin master changes를 체크하는 yml 생성
  • 매일 밤 0시 체크 cron 및 이전 pr 체크 후, 이전 커밋 pr 발생시 기존 pr 자동 close.

Copy link
Member

@chaaerim chaaerim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오 좋은데요!! 감사합니다 👍
근데 이전 pr 체크 후, 이전 커밋 pr 발생시 기존 pr 자동 close. 요 말이 잘 이해가 안되어서 설명 추가로 해주실 수 있나요!?

@minchodang
Copy link
Member Author

@chaaerim
앗 넵 제가 말을 모호하게 적었네요. ㅎㅎ
이전 pr 체크 후, 이전 커밋 pr 발생시 기존 pr 자동 close.

이 부분 설명 드리자면, 혹여나 발생할 우려 사항을 방지하기 위한 안전 장치 정도입니다.
상황을 들어볼게용.

만약, 기존에 이 action에 의해 열렸던 pr이 병합되지 않았는데.
갑자기 upstream 레포 master에 새로운 changes 커밋이 올라온다고 가정해 볼게요.

그런데, 그때 우연하게도 다시 action이 돌아서 pr 생성이 될 수 있습니다.

그렇다면 결론적으로 불 필요한 기존 pr이(우리가 병합하지 않았던 이전 sync 업스트림 커밋들로 구성된) 이미 하나 있는데도 불구하고, 이 액션으로 인해서 총 2개의 pr이 생기게 되겠죠?

그래서 그럴 바에는 기존 pr들 체크해서 현재 upstream 커밋 해시값 체크해서 동일하면 현재 열려있는 pr 유지하고, 아니면 닫습니다.
이후, 최신화된 커밋으로 구성된 pr을 다시 생성합니다.

이런 식으로 하면, 결국 우리가 혹여 upstream 레포와 병합 시기를 놓쳤다고 하더라도, 이 액션에 따른 sync 맞추는 pr은 하나만으로 유지가 될 겁니다.

혹시 이해가 안 되셨다면 다시 알려주세용. ㅎㅎ

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.

2 participants