-
Notifications
You must be signed in to change notification settings - Fork 1
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
[iOS] Workspace & CI 적용 #129
[iOS] Workspace & CI 적용 #129
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.
확인했습니다. 라이브러리 명을 바꾸셨던데 맞나요?
Feature 패키지 구조도 컨밴션을 정해놓으면 좋을 것 같네요.
❗ 배경
close #128
🔧 작업 내역
🧪 테스트 방법
CI로 테스트합니다.
📝 리뷰 노트
Feature 네이밍
Feature 네이밍이 다 제각각이더라구요..!
그래서 동사+명사 혹은 명사로 패턴을 일치시켜줬습니다.
최대한 수정이 없는 형태로 설정했는데 그래도 아마 몇 가지는 바뀌었을 것 같아요..!
일단 제가 확인한 패키지는 두 가지 입니다..