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

[Spring 체스 - 1단계] 레넌(조형래) 미션 제출합니다. #393

Merged
merged 21 commits into from
Apr 25, 2022

Conversation

brorae
Copy link

@brorae brorae commented Apr 25, 2022

안녕하세요,스티치!
도메인 코드를 수정하지 않고 스프링으로 전환하도록 노력하였습니다.
리뷰 잘 부탁드립니다!

Copy link

@lxxjn0 lxxjn0 left a comment

Choose a reason for hiding this comment

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

레넌, 안녕하세요 :) 이번에 레넌의 리뷰를 맡게 된 스티치입니다 😄

이번 1단계 리뷰어 가이드를 봤는데 1단계는 스프링 애플리케이션의 적용에 대한 부분만 빠르게 리뷰하고 2단계에서 상세 리뷰를 진행하기를 제안해주셨더라구요. 그래서 이번 1단계에서는 스프링 부분에 대한 것만 리뷰를 남겨두도록 할게요. 양해 부탁드리고 상세한 리뷰는 2단계 때 이어서 진행하도록 하겠습니다 👍🏼

전체적으로 스프링 애플리케이션 적용을 잘 해주신 것 같아요!! 그래서 남겨준 리뷰 말고는 추가로 수정할 부분은 없는 것 같습니다!! 빠르게 머지하고 2단계에서 상세 리뷰 진행하도록 할게요 :)

추가로 스프링의 빈 주입에 대한 부분을 한번 공부해보길 추천드릴게요!! 다양한 방법이 있는데 현재 코드에서는 중복된 방법으로 사용되는 경우도 있는 것 같아요!! 각각의 장단점도 함께 공부하면 좋을 것 같네요 :)

레넌, 2단계에서 뵐게요 ✋🏼

@lxxjn0 lxxjn0 merged commit dd5d7b1 into woowacourse:brorae Apr 25, 2022
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