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

job이 서로 의존성 있게 수정 #573

Open
unluckyjung opened this issue Oct 4, 2021 · 1 comment
Open

job이 서로 의존성 있게 수정 #573

unluckyjung opened this issue Oct 4, 2021 · 1 comment
Labels
back-end 백엔드 크루가 진행할 작업 infra 인프라 관련 priority-3 우선 순위(숫자가 작을 수록 중요도가 높다.) refactor 리팩토링 대상

Comments

@unluckyjung
Copy link
Collaborator

저희의 경우 #368 에서 언급한 빌드 순서를 보장 받아야 하는 이유때문에, 병렬적으로 도는 job을 커버치기 위해서 한개의 job에 모든것을 때려넣은 형태입니다. 현재 yml

그런데 github ci/cd 를 복습하다보니 문서처럼 의존성을 부여해서 순서를 정해둘 수 있는데 현재 저희는 한 job에 다 때려박았더라구요.

그래서 needs의 존재를 몰랐나? 하고 pr을 뒤져보니 이때는 needs로 잘 구성했는데, 저번에 yml 대공사할때, 그냥 한개의 job에서 퉁치게 해두었네요.

혹시 현재처럼 구성한 특별한 이유가 있는게 아니라면, job들을 분리해보는게 어떨까 하여 이슈 남겨 놓습니다.

@unluckyjung unluckyjung added refactor 리팩토링 대상 back-end 백엔드 크루가 진행할 작업 priority-3 우선 순위(숫자가 작을 수록 중요도가 높다.) infra 인프라 관련 labels Oct 4, 2021
@unluckyjung unluckyjung added this to the 마지막 데모데이 milestone Oct 4, 2021
@Hyeon9mak
Copy link
Collaborator

오잉! 제가 제대로 파악한게 맞다면

ae69370#diff-87ee5504a3e25ac558b343724c905f2f7949e8cec3d92b9c4300bb922afa164f

PR에 의해서 needs 사용이 사라진거 같아요

@Hyeon9mak Hyeon9mak removed this from the 마지막 데모데이 milestone Dec 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
back-end 백엔드 크루가 진행할 작업 infra 인프라 관련 priority-3 우선 순위(숫자가 작을 수록 중요도가 높다.) refactor 리팩토링 대상
Projects
None yet
Development

No branches or pull requests

2 participants