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.
관련 이슈
작업 내용
작업 과정을 모두 정리해두었습니다.
https://celestial-sheep-6b7.notion.site/b3f5b05149544e72b15ed6f2999bf6f7
특이 사항
머지가 되어야 깃허브 액션이 잘 동작하는지 확인할 수 있기 때문에
머지 후 더 수정해야 할 수도 있습니다😓
리뷰 요구사항 (선택)
서브모듈을 적용하면 할 수록 단점들이 보이더라고요..
서브모듈에 대한 러닝커브
사실 러닝커브가 높진 않은데, 제가 괜히 욕심을 부려 새로운 기능을 구현하시는 새원님께 공부할걸 드린건 아닌지.. 하는 생각이 들었습니다.
서브모듈이 빌드,배포할 때도 잘 적용되는지 보려면 “깃허브 액션을 돌려야 한다.”
그런데 깃허브 액션에 realse 브랜치에 머지가 될 때 돌아가도록 설정이 되어있습니다. 그래서 지금 이걸 바꾸더라도, ‘잘 배포가 되는지’ 확인하려면 realse 브랜치에 main 브랜치를 머지를 한 뒤여야 할 것입니다. 아마 모든 기능을 구현한 이후이겠지요? 이처럼 확인을 하기까지 오래 걸린다는 점이 있습니다.
사실 yml 공유만 열심히 하기만 하면 된다.
사실 yml 파일을 잘 공유하기만 하면 해결되는 문제라는 생각도 드네요.. 세원님이 주신 application-secret 파일을 적용하니 잘 돌아갔습니다.
그래서 장단점을 비교해봤을 때 개인적으로는 단점이 더 크게 느껴졌습니다. 서브모듈을 도입해서 얻을 수 있는 장점은 1) 민감한 정보 숨기기 2) yml 파일 버전관리하기 정도 3) 새로운 기술에 대한 호기심 해소 정도인데, 1번은 우리 레포 자체가 원래 private 이라 큰 이점은 없고요.. 3번은 지금의 우선순위가 아니니 적용했을 때의 수고로움이 장점보다 더 크지 않나 하는 생각이 듭니다😓
우선 제가 구현은 해두긴 했지만 세원님이 장단점을 살펴보시고 merge 하셔도 되고 / 안하셔도 됩니다! 세원님이 결정하시는대로 따르겠습니다.