Skip to content

[최호] 15장,16장,17장 : JUnit 들여다보기, SerialDate 리팩터링, 냄새와 휴리스틱 #38

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.

Sign up for GitHub

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

Jump to bottom
Merged
bbbang105 merged 3 commits into from
Sep 7, 2025
Merged

[최호] 15장,16장,17장 : JUnit 들여다보기, SerialDate 리팩터링, 냄새와 휴리스틱 #38

bbbang105 merged 3 commits into from
Sep 7, 2025

Conversation

@choihooo
Copy link
Collaborator

@choihooo choihooo commented Aug 24, 2025

No description provided.

bbbang105
bbbang105 approved these changes Aug 24, 2025
View reviewed changes
Copy link
Member

@bbbang105 bbbang105 left a comment

Choose a reason for hiding this comment

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

책 다 읽느라 고생하셨습니다 👍🏻👍🏻👍🏻

15장/최호.md Outdated
Comment on lines +17 to +22
Copy link
Member

@bbbang105 bbbang105 Aug 24, 2025

Choose a reason for hiding this comment

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

굿굿!!

Copy link
Member

@yeyounging yeyounging Aug 25, 2025

Choose a reason for hiding this comment

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

저도 저 토스 문서 너무 좋더라구요. 👍

16장/최호.md Outdated
Copy link
Member

@bbbang105 bbbang105 Aug 24, 2025

Choose a reason for hiding this comment

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

음 좋네요 만나서 얘기해봅시다!

17장/최호.md Outdated
Copy link
Member

@bbbang105 bbbang105 Aug 24, 2025

Choose a reason for hiding this comment

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

저도 시간이 없을 때는 마지막 장이라도 다시 읽어보려구요

yeyounging
yeyounging approved these changes Aug 25, 2025
View reviewed changes
15장/최호.md Outdated
Comment on lines +17 to +22
Copy link
Member

@yeyounging yeyounging Aug 25, 2025

Choose a reason for hiding this comment

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

저도 저 토스 문서 너무 좋더라구요. 👍

17장/최호.md Outdated
Copy link
Member

@yeyounging yeyounging Aug 25, 2025

Choose a reason for hiding this comment

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

특히 ai가 나오면서 가끔은 기능 구현에 오히려 더 급급해진 느낌이 들어요. 프로젝트 자체가 커다란 알고리즘이라 생각하고 생각하는 습관이 필요한 것 같습니다. ㅎㅎ

@bbbang105 bbbang105 merged commit 33d3920 into main Sep 7, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bbbang105 bbbang105 bbbang105 approved these changes

@yeyounging yeyounging yeyounging approved these changes

@karnelll karnelll Awaiting requested review from karnelll karnelll is a code owner

@codrin2 codrin2 Awaiting requested review from codrin2 codrin2 is a code owner

@HoyeongJeon HoyeongJeon Awaiting requested review from HoyeongJeon HoyeongJeon is a code owner

@nebulaBdj nebulaBdj Awaiting requested review from nebulaBdj nebulaBdj is a code owner

Assignees

@choihooo choihooo

Labels

15장 16장 17장

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@choihooo @bbbang105 @yeyounging