-
Notifications
You must be signed in to change notification settings - Fork 6
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
Infra: 프론트엔드 개발 서버 CI/CD 구축 #6
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
와 정말 프로페셔널 해보여요 👍 잘..모르는 부분이라 동호님을 믿고, 다른 분들의 리뷰를 기다립니다..!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
리뷰 이벤트 참여합니다🤣✨
LGTM👍👍
프론트까지 정말 감사합니다!!🙏
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
감사합니당 동호님~~ LGTM👍👍👍🔥🔥🔥
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM👍👍고생하셨습니다~~!! 상품은 치킨인가요?
치킨은 오늘 제가 맛있게 먹도록 하겠습니다 |
|
||
- name: S3 and CloudFront Deploy | ||
uses: Reggionick/s3-deploy@v4 | ||
with: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
어쩌다 CI/CD 관련 글을 보다가, 다시 한 번 옵션들을 살펴보았습니다! 여기에서 delete-removed: true
옵션은 필요 없을까요??
"delete-removed: Removes files in S3, that are not available in the local copy of the directory" - Reggionick/s3-deploy
"로컬에서 삭제한 파일이 S3에서도 자동으로 삭제되므로, 불필요한 파일을 자동으로 정리할 수 있습니다. 하지만 주의하여 사용해야 합니다. 실수로 중요한 파일을 삭제하지 않도록 주의해야 합니다." - Chat GPT
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
제가 이해하기로 해당 옵션이 Github Repository(local)에서 지운 파일에 대해서 S3에도 동일하게 제거하는 옵션인건가요??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
네! 저도 그렇게 이해했습니다~!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
그러면 아무래도 옵션을 키는게 좋겠네요! 감사합니다 👍👍👍
개요
dev
브랜치에 해당하는, 프론트엔드 개발 서버의 CI/CD 구축 완료 했습니다!보안을 강화하거나 성능 개선 할 여지가 있지만 둘 다 차차 진행하도록 하겠습니다!
작업 사항
참고 사항 (optional)
이건 불편함을 느끼실 때 다시 작업하는 걸로 하겠습니다!
관련 이슈 (optional)
스크린샷
리뷰어에게