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: 프로젝트의 전반적인 리팩토링 #84

Merged
merged 52 commits into from
Aug 14, 2024

Conversation

guesung
Copy link
Collaborator

@guesung guesung commented Aug 13, 2024

What is this PR? 🔍

  • 프로젝트의 전반적인 리팩토링

Changes 📝

  • 양이 너무 많아서, 커밋명으로 확인해주시면 감사하겠습니다.
  • 컨벤션에 맞지 않는 부분들을 수정 및 이전에 제가 작성한 코드보다 더 효율적이고 직관적인 코드라는 생각이 드는 부분들을 수정하였습니다. 궁금하신 점, 의문가는 점, 이해안되는 부분 전부 편하게 말씀해주세요 !!

Screenshot 📷

@guesung guesung merged commit 6b1ab23 into develop Aug 14, 2024
1 check passed
@guesung guesung deleted the sprint/refactor-the-project branch August 14, 2024 04:37
seondal added a commit that referenced this pull request Aug 16, 2024
@seondal seondal changed the title Refactor: 프로젝트의 전반적인 리팩토링 Refactor: 프로젝트의 전반적인 리팩토링 (수정 필요) Aug 16, 2024
@seondal seondal changed the title Refactor: 프로젝트의 전반적인 리팩토링 (수정 필요) Refactor: 프로젝트의 전반적인 리팩토링 -> 코멘트 확인해주세요! Aug 16, 2024
@seondal seondal changed the title Refactor: 프로젝트의 전반적인 리팩토링 -> 코멘트 확인해주세요! Refactor: 프로젝트의 전반적인 리팩토링 -> 규성님, 코멘트 확인해주세요! Aug 16, 2024
@guesung guesung changed the title Refactor: 프로젝트의 전반적인 리팩토링 -> 규성님, 코멘트 확인해주세요! Refactor: 프로젝트의 전반적인 리팩토링 Aug 24, 2024
@dnd-side-project dnd-side-project deleted a comment from seondal Aug 24, 2024
@dnd-side-project dnd-side-project deleted a comment from seondal Aug 24, 2024
@dnd-side-project dnd-side-project deleted a comment from seondal Aug 24, 2024
@dnd-side-project dnd-side-project deleted a comment from seondal Aug 24, 2024
@dnd-side-project dnd-side-project deleted a comment from seondal Aug 24, 2024
@guesung guesung removed their assignment Aug 24, 2024
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.

1 participant