Refactor/#110 remove unused comments logger#114
Refactor/#110 remove unused comments logger#114zzangzzangguy wants to merge 8 commits intodevelopfrom
Conversation
0Hooni
left a comment
0Hooni
left a comment
There was a problem hiding this comment.
전반적으로 안쓰이는 코드들을 제거하는 작업을 하신것 같네요!
몇몇 파일들에 대한 의문이 좀 있어서 코멘트 남겨놨습니다. 답변 주시고 리뷰 재요청 눌러주시면 될 것 같아요!
수고하셨습니다 ㅎㅎ
| fastlane/Preview.html | ||
| fastlane/screenshots/**/*.png | ||
| fastlane/test_output | ||
| Poppool/Poppool/Infrastructure/*.mobileprovision |
There was a problem hiding this comment.
혹시 프로비저닝이 프로젝트 폴더 안에 있는건가요?
일단 프로비저닝은 프로젝트 폴더 외부에 있어도 등록가능하기에 폴더에 넣는것보다 외부에 두는것을 권장드립니다!
아마 조만간 모듈화 들어가면 까먹으셨다가 또 프로비저닝 들어가십니다 🥲
외부에 두셨다면 ignore 세팅 지워주시기까지 하면 감사하겠습니다 🙇🏻♂️
There was a problem hiding this comment.
브랜치 분리 이후 에 작업해둔거군뇨.. 외부에 두도록 하겠습니다 😂
There was a problem hiding this comment.
@zzangzzangguy 이번 PR에서 제거해주세요!
이번 PR에 들어가버리면 나중에 까먹고 남아있을것 같아서🥲
Poppool/File.txt
Outdated
There was a problem hiding this comment.
해당화면 디버깅중에 잠깐 주석처리해둔 부분인데 필요치않은것 같아 다음 PR에 제거하겠습니다
There was a problem hiding this comment.
@zzangzzangguy 어차피 지금 PR이 주석 제거니까 그냥 여기서 지워도 될것 같아요!
There was a problem hiding this comment.
레거시코드.. ㅜ 제거하도록 하겠습니다
There was a problem hiding this comment.
안쓰는 코드는 꼭 삭제....!!!
전체 주석이 아마 나중에 쓰려고 냅두는거같은데, 개인적으로 좋지 않은 습관이라고 생각해요...🥲
사용이 필요하다면 사용할 때 구현하시고, 나중에 쓰시려 하시는 거라면 팀의 레포지토라가 아닌 개인 폴더로 관리하시는것도 좋을것 같네욥
There was a problem hiding this comment.
아 기존 해당방식으로 MapViewController 를 분리하려 하다 브랜치분리 이후 제거된 코드인데 올라갔군요 ! 제거되었습니다
There was a problem hiding this comment.
@zzangzzangguy 지금 브랜치 뎁스가 어떻게 되어있는건지 알 수 있을까요?
지금 PR에서는 제거된 커밋이 안올라온것 같아요!
| } | ||
|
|
||
| private func showNoMarkersToast() { | ||
| // 디자인 예정이므로 임시 구현 |
There was a problem hiding this comment.
이런 주석은 일반 주석보다는 TODO 주석을 이용해보는것도 방법입니다!
// TODO: 할 일이렇게 사용하시면 됩니다👍🏻
dongglehada
left a comment
dongglehada
left a comment
There was a problem hiding this comment.
영훈님이 상세하게 확인을 해주셔서 저는 코멘트 하나 남긴 부분말고는 확인하였습니다!
There was a problem hiding this comment.
해당 옵저버를 사용하지 않는 것 같은데 지워도 괜찮지 않을까요.?
There was a problem hiding this comment.
To DO로 변경되었습니다!
3 participants
📌 이슈
✅ 작업 사항
기타 불필요한 주석과 로그 삭제