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

[Remove] 미사용 함수 및 변수 삭제 #98

Merged
merged 23 commits into from
Mar 22, 2024
Merged

[Remove] 미사용 함수 및 변수 삭제 #98

merged 23 commits into from
Mar 22, 2024

Conversation

Jeongmin39
Copy link
Contributor

✨ PR 유형

어떤 변경 사항이 있나요??

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

🛠️ 작업내용

사용하지 않는 함수 및 변수, 파일 삭제
미사용 import문 삭제
오타 수정

📋 추후 진행 상황

place 조회 필터 적용 시 생기는 문제를 해결할 예정입니다.

📌 리뷰 포인트

수정 부분은 많지만 코드에 영향을 주지 않는 선에서 건들였습니다.
간단한 변경사항들이니 한 번씩들 확인 부탁드려요!

✅ Checklist

PR이 다음 요구 사항을 충족하는지 확인해주세요

  • 커밋 메시지 컨벤션에 맞게 작성했습니다
  • 유지-보수를 위해 주석 처리를 잘 작성했습니다

@Jeongmin39 Jeongmin39 self-assigned this Mar 22, 2024
Copy link
Contributor

@choeun7 choeun7 left a comment

Choose a reason for hiding this comment

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

와 오타 수정까지,, 진짜 꼼꼼하게 확인해주셨네요 고생하셨습니다!!

@choeun7 choeun7 merged commit b12d55e into dev Mar 22, 2024
2 checks passed
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.

2 participants