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

[FE] ci: concurrency의 cancel-in-progress 설정을 false로 수정 #448

Merged
merged 1 commit into from
Aug 20, 2024

Conversation

ImxYJL
Copy link
Contributor

@ImxYJL ImxYJL commented Aug 20, 2024


🚀 어떤 기능을 구현했나요 ?

  • 기존 PR에 push가 일어나 CI가 돌고 있을 때, 다른 PR의 push가 일어나면 기존 PR의 CI 작업이 취소되는 일을 막기 위한 작업을 했습니다.

🔥 어떻게 해결했나요 ?

  • CI 설정 중 concurrency(동시성) 부분에서 cancel-in-progress (새 작업이 생기면 기존 작업 취소)가 true였는데,
  • 기본값이 false라 그냥 이 concurrency 부분을 삭제해버리려고 했는데 명시적으로 작성하는 게 나을 것 같아 false로 변경했습니다.

📝 어떤 부분에 집중해서 리뷰해야 할까요?

  • 기본값이 false니까 concurrency 부분을 아예 삭제해버릴까요?
  • 해당 속성이 비교적 직관적이지 않은 것 같아 무슨 작업에 대한 설정인지를 설명하는 주석을 달아뒀었는데 이 주석이 필요할까요?

📚 참고 자료, 할 말

  • 위 속성은 혼자 미션할 때는 유용했는데 이렇게 팀 협업을 할 때는 세세한 커스텀이 들어갈 게 아니라면 안 하는 게 나은 것 같습니다.

@ImxYJL ImxYJL added this to the 4차 스프린트: 최종 milestone Aug 20, 2024
@ImxYJL ImxYJL self-assigned this Aug 20, 2024
@ImxYJL ImxYJL linked an issue Aug 20, 2024 that may be closed by this pull request
@ImxYJL ImxYJL changed the title ci: concurrency의 cancel-in-progress 설정을 false로 수정 [FE] ci: concurrency의 cancel-in-progress 설정을 false로 수정 Aug 20, 2024
Copy link
Contributor

@soosoo22 soosoo22 left a comment

Choose a reason for hiding this comment

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

또 하나 배워갑니다! 올리!!

Copy link
Contributor

@chysis chysis left a comment

Choose a reason for hiding this comment

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

기본값을 사용하긴 하지만 cancel-in-progress를 명시해도 상관없을 것 같아요!
고생했어요🔥

Copy link
Contributor

@BadaHertz52 BadaHertz52 left a comment

Choose a reason for hiding this comment

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

꿋!!! 👍

@BadaHertz52 BadaHertz52 merged commit 09529da into develop Aug 20, 2024
4 checks passed
@donghoony donghoony deleted the fe/fix/446-ci-concurrency branch August 23, 2024 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[FE] CI의 concurrency 속성을 제거한다.
4 participants