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

[REFACTOR] 불필요한 에셋 정리 & SwiftLint 수정 & README 추가 #314

Merged
merged 9 commits into from
Jan 23, 2022

Conversation

mini-min
Copy link
Member

🌴 PR 요약

🌱 작업한 브랜치

🌱 작업한 내용

  • README 기본적인 내용들 추가했습니다..나중에 기능별로 더 올려볼게요....군대가기전..언젠가..
  • Assets 파일에서 사용하지 않는 것들 삭제했습니다. (조금이나마..용량에 도움이될까해서요..)
  • 예전 코리뷰에서 현규 교수님이 말하셨던 Swiftlint file_length 풀어두었습니다..! 리팩토링할때 파일 길이 400자 안넘게 코드짜는걸로...! 일단 지금 400자 넘는것들은 제가 임의로 수정해놨습니다 ^__^
  • 회원가입 서버 통신에서 더이상 사용하지 않는 부분들 삭제했습니다.
  • Cell register 부분에 Const로 지정되어 있지 않는 부분 수정했습니다.

📮 관련 이슈

@mini-min mini-min added Minjae 🐻‍❄️ 민재 강남1타강사 작업 Refactor 로직은 변경하지 않고, 기존의 코드를 개선하는 리펙토링 labels Jan 20, 2022
@mini-min mini-min self-assigned this Jan 20, 2022
@mini-min mini-min merged commit 92821e3 into TeamNADA:main Jan 23, 2022
@mini-min mini-min deleted the release/#313 branch January 23, 2022 12:56
Comment on lines +324 to +330
case 1: defaultImageIndex = 1
case 2: defaultImageIndex = 2
case 3: defaultImageIndex = 3
case 4: defaultImageIndex = 4
case 5: defaultImageIndex = 5
case 6: defaultImageIndex = 6
case 7: defaultImageIndex = 7
Copy link
Member

Choose a reason for hiding this comment

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

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Minjae 🐻‍❄️ 민재 강남1타강사 작업 Refactor 로직은 변경하지 않고, 기존의 코드를 개선하는 리펙토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[REFACTOR] 불필요한 에셋 정리 & SwiftLint 수정 & README 추가
2 participants