search

cloud-barista / cb-operator

The Operator for Cloud-Barista system
Apache License 2.0
6 stars 6 forks source link

Add build-test GitHub workflow [ci skip test] #158

Closed jihoon-seo closed 3 years ago

jihoon-seo commented 3 years ago

@hermitkim1 리뷰 감사합니다 😊 제안해 주신 사항들을 모두 반영했습니다. Go version을 명시적으로 적으니 좋네요.. ㅎㅎ

[skip ci] 관련해서는, 지금 상태로는 commit 메시지에 [skip ci] 가 적혀있어야 하는 것 같은데, GitHub PR의 제목에 [skip ci] 를 적어도 적용이 되면 편할 것 같습니다.. 😊

yunkon-kim commented 3 years ago

/lgtm

yunkon-kim commented 3 years ago

@hermitkim1 리뷰 감사합니다 😊 제안해 주신 사항들을 모두 반영했습니다. Go version을 명시적으로 적으니 좋네요.. ㅎㅎ

[skip ci] 관련해서는, 지금 상태로는 commit 메시지에 [skip ci] 가 적혀있어야 하는 것 같은데, GitHub PR의 제목에 [skip ci] 를 적어도 적용이 되면 편할 것 같습니다.. 😊

@jihoon-seo 넵, GitHub에서 말씀하신 두가지 케이스를 모두 커버할 수 있도록 신경을 쓴것 같습니다 ^^

jihoon-seo commented 3 years ago

@hermitkim1 관련하여 테스트를 해 보았습니다.

https://github.com/cloud-barista/cb-tumblebug/pull/733

그랬더니 CI 가 skip되지 않고 실행되었습니다. 😢 https://github.com/cloud-barista/cb-tumblebug/runs/3594256520

GitHub PR의 제목에 [skip ci] 를 적어도 적용

--> https://bugkingk.github.io/2020-12-12-post-27/ 를 참고하여 적용하면 될 것 같습니다. 😊

jihoon-seo commented 3 years ago

[GitHub PR의 제목에 [skip ci] 적으면 CI skip하도록 설정]

    if: >-
      ${{ !contains(github.event.pull_request.title, 'skip ci') &&
      !contains(github.event.pull_request.title, 'ci skip') &&
      !contains(github.event.pull_request.title, 'no ci') &&
      !contains(github.event.pull_request.title, 'skip actions') &&
      !contains(github.event.pull_request.title, 'actions skip')
      }}

[테스트]

[PR 제목 바꾸기 전]

[PR 제목 바꾼 뒤]

PR 제목에 skip ci, ci skip, no ci, skip actions, actions skip 중 하나를 넣으면 Skip CI 되는 것을 확인했습니다. 😊

seokho-son commented 3 years ago

LGTM!