Skip to content

Conversation

@nayonsoso
Copy link
Collaborator

@nayonsoso nayonsoso commented Dec 7, 2024

작업 내용

컨벤션이 통일되지 않아 인지부하가 오는 부분을 줄이기 위해 컨벤션을 통일합니다.

  • 안 쓰는 함수, 클래스 삭제
  • 의미없는 주석 삭제
  • 개행 통일
  • 와일드카드(*) import 삭제

특이 사항

코드 내용 상으로 변경된 부분들은 거의 없습니다.
안 쓰는 dto 삭제한 정도입니다.

오랜만에 코드를 보니 읽기가 힘들더라고요😓
왜일까 생각해보니 컨벤션이 통일되어있지 않은게 1차적인 원인이었습니다.
추후 참여할 개발자들도 저와 같은 것을 느낄 것이니,
앞으로의 유지보수를 위해서 컨벤션 통일하는 것이 좋겠습니다.

- post 패키지에서만 쓰이는 dto를 post.dto 안으로 이동
- 주석 삭제
- 함수 나열 순서 통일
- 개행 통일
- asterisk(*) 제거
- 개행 통일
@nayonsoso nayonsoso requested a review from wibaek December 7, 2024 15:34
@nayonsoso nayonsoso self-assigned this Dec 7, 2024
@leesewon00 leesewon00 requested review from leesewon00 and removed request for wibaek December 9, 2024 11:08
Copy link
Member

@leesewon00 leesewon00 left a comment

Choose a reason for hiding this comment

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

LGTM

@nayonsoso nayonsoso merged commit cfcda24 into main Dec 9, 2024
@nayonsoso nayonsoso deleted the style/conevtion branch February 15, 2025 19:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants