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

[Chore] #57 - 일러스트 변경 및 린트 룰 수정 #58

Conversation

lsj8706
Copy link
Collaborator

@lsj8706 lsj8706 commented Jan 9, 2023

🌱 작업한 내용

  • 바뀐 이미지 에셋으로 교체
  • try! 문을 쓸 수 있도록 force_try 를 린트 룰에서 제거

🌱 PR Point

  • 새로운 이미지 에셋으로 교체하고 필요한 부분 레이아웃도 수정했기 때문에 별도로 이미지 관련해서 추가 작업은 필요 없습니다!
  • 앞으로 서버 통신 코드를 짤 때 Router에서 try! [필요한 Dto].asParameter() 를 사용할 수 있도록 force_try를 린트 룰에서 해제했습니다. 일단 해제는 했으나 당연히 !를 통한 강제 시도는 하지 않는 것이 좋습니다 가능 하다면 그냥 try를 쓰고 do ~ catch문으로 감싸서 구현하는 것이 좋습니다.

📸 스크린샷

  • 생략

📮 관련 이슈

@lsj8706 lsj8706 added Chore 그 외의 잡일/버전 코드 수정, 패키지 구조 변경, 파일 이동, 파일 이름 변경, 레이아웃 조정 등 세진☃️ labels Jan 9, 2023
@lsj8706 lsj8706 self-assigned this Jan 9, 2023
Copy link
Contributor

@dlwogus0128 dlwogus0128 left a comment

Choose a reason for hiding this comment

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

팀짱님 짱~~~~~~~~₩

@lsj8706 lsj8706 merged commit 8db8178 into Runnect:develop Jan 9, 2023
@lsj8706 lsj8706 deleted the chore/#57-일러스트-변경-및-린트를수정 branch January 9, 2023 06:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Chore 그 외의 잡일/버전 코드 수정, 패키지 구조 변경, 파일 이동, 파일 이름 변경, 레이아웃 조정 등 세진☃️
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[Chore] 일러스트 변경 및 린트 룰 수정
2 participants