[FEAT/#22] CI를 진행합니다. #23
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Kiyoung-Kim-57
approved these changes
Nov 12, 2024
| override func viewDidLoad() { | ||
| super.viewDidLoad() | ||
| view.backgroundColor = .red | ||
|
|
0Hooni
reviewed
Nov 13, 2024
Collaborator
0Hooni
left a comment
0Hooni
left a comment
There was a problem hiding this comment.
CI 만들어주셨군요!!
감사합니다. 수고 많으셨어요👍
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🤔 배경
main과 develop 브랜치는 많은 작업 브랜치의 베이스가 되는 중요한 브랜치입니다.
그래서 main과 develop 브랜치에 PR이 머지되기 전 / 머지된 후 빌드가 정상적으로 되는지 확인하기 위한 작업을 자동화하기 위해 Github Action을 활용해 빌드를 자동화했습니다.
📃 작업 내역
✅ 리뷰 노트
빌드가 되는지를 테스트하는건데 DerivedData를 캐싱해도 되는지 의문입니다 🤔
DerivedData를 통해 증분 빌드를 하게 될텐데, 완전히 깨끗한 환경에서 빌드가 되는지를 테스트해야하는지 고민이네요?!
우선 SPM Dependency와 DerivedData를 캐싱하지 않았을 때와 캐싱 했을 때 CI 속도는 각각 8분 vs 1분 50초로 어마어마한 차이가 났습니다..
무결한 환경에서의 빌드 테스트 vs 무결성은 살짝 떨어지지만 빠른 빌드
뭐가 좋을까요?
🎨 스크린샷