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

[#18] 서버 배포 #36

Merged
merged 2 commits into from
Aug 16, 2021
Merged

[#18] 서버 배포 #36

merged 2 commits into from
Aug 16, 2021

Conversation

juyoungpark718
Copy link
Collaborator

@juyoungpark718 juyoungpark718 commented Aug 16, 2021

관련이슈

실제 소요시간

4h

체크리스트

  • base branch를 확인했나요?

설명

  • 개발 서버에 대한 github actions 스크립트입니다.
  • 별 의미는 없지만 diff를 적용해봤습니다.
    • 현재는 서버, 클라이언트에서 바뀐 코드가 있는 경우만 빌드하는 방식입니다.
  • 클라이언트의 경우 actions 서버에서 build된 파일을 개발서버에 옮기는 방식입니다
    • 예전에 겪었던 이슈 중에, build가 오래걸려서 pr이 여러번 겹치는 경우 개발서버가 버티지못하는 경우가 있었습니다. 그래서 PR이 여러개 겹치더라도, 복사된 파일을 붙여 넣기 때문에 빌드 시간이 길어져도 github actions에서 처리되고, build된 파일을 붙여넣기 때문에 github actions가 끝나는 시간이 보장만 된다면 배포가 잘 이루어져서 해당 방법을 사용했습니다.
  • ts->js로 변환 후 pm2 reload를 하기 때문에 package.json에 build 스크립트를 추가했습니다.

@juyoungpark718 juyoungpark718 linked an issue Aug 16, 2021 that may be closed by this pull request
4 tasks
Copy link
Member

@Choi-Jinwoo Choi-Jinwoo left a comment

Choose a reason for hiding this comment

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

오 비교하는 부분까지 하셨군요 !

수고하셨습니당 :)

@sanginchun
Copy link
Collaborator

고생하셨습니다! 👍

@juyoungpark718 juyoungpark718 merged commit cdc080d into develop Aug 16, 2021
@Choi-Jinwoo Choi-Jinwoo deleted the feat/server-deploy branch August 23, 2021 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend 백엔드 feature 기능
Projects
None yet
Development

Successfully merging this pull request may close these issues.

서버 배포
3 participants