Skip to content
New issue

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

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

Already on GitHub? # to your account

2-6 [배포] [fix] checkout step 추가 #46

Merged
merged 1 commit into from
Nov 22, 2022
Merged

Conversation

leesungbin
Copy link
Member

개요

https://github.com/dorny/paths-filter 에서 actions/checkout step이 필요없다고 이야기한 부분on - pull request 일 때 입니다.
현재 자동 배포에 사용하고 있는 action은 on - push 일 때여서, 해당 스텝이 필요했으나, #39 에서 누락하였습니다.

작업사항

  • actions/checkout step 추가
  • private repo에서 해당 액션 테스트

리뷰 요청사항

  • 자동 배포가 정상적으로 작동하기 위해서는 이 PR이 우선적으로 merge 되어야합니다.

`https://github.com/dorny/paths-filter` 에서 `actions/checkout` step이 필요없다고 이야기한 부분은 on - pull request 일 때 이다.
push 일 때는 필요하다.
@leesungbin leesungbin marked this pull request as ready for review November 21, 2022 14:31
@github-actions github-actions bot requested review from JunYupK and Palwol November 21, 2022 14:31
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants