Skip to content

[FEATURE] 리팩터링 챕터 9 작성 완료 #18

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
s3ich4n merged 22 commits into from
Mar 17, 2025
Merged

[FEATURE] 리팩터링 챕터 9 작성 완료 #18

s3ich4n merged 22 commits into from
Mar 17, 2025

Conversation

@s3ich4n
Copy link
Owner

@s3ich4n s3ich4n commented Mar 17, 2025 •
edited
Loading

Summary

리팩터링 챕터 9 정리 완료 (resolves #17)

Changes

  • 책의 내용 1회독
  • 테스트코드 추가
  • 캡슐화의 다른 언어에 대한 명확성과, 파이썬의 특징 소개

PR Checklists

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

  • 변경 사항에 대한 테스트를 했습니다(버그 수정/기능에 대한 테스트).

s3ich4n added 22 commits March 17, 2025 23:00
@s3ich4n
feat(chapter09): add chapter on data organization and refactoring pri…
83dcfaa 
…nciples
@s3ich4n
feat(chapter09): include chapter09 in test and coverage configurations
5087a94
@s3ich4n
feat(chapter09): implement distance calculation and add tests for sce…
407cfe6 
…nario handling
@s3ich4n
refactor(chapter09): improve variable naming for clarity in distance …
9666983 
…calculation
@s3ich4n
refactor(chapter09): improve variable naming for clarity in distance …
af6d938 
…calculation
@s3ich4n
refactor(chapter09): rename variable for secondary acceleration in ca…
46a38e8 
…lculation
@s3ich4n
refactor(chapter09): distance calculation into primary and secondary …
5ce66c8 
…functions
@s3ich4n
feat(chapter09): add discount function with parameterized tests
db00c54
@s3ich4n
refactor(chapter09): simplify discount function by using result variable
8799941
@s3ich4n
feat(chapter09): add Organization class for encapsulating organizatio…
3c7215c 
…n data
@s3ich4n
refactor(chapter09): rename Organization class attribute from name to…
ee60ac3 
… title
@s3ich4n
feat(chapter09): implement SomeClass for production adjustments with …
5c42dbe 
…tests
@s3ich4n
refactor(chapter09): add calculated_production property for productio…
701c9b4 
…n adjustments
@s3ich4n
refactor(chapter09): remove calculated_production property and relate…
8b30fb1 
…d tests
@s3ich4n
refactor(chapter03): remove direct production adjustment in apply_adj…
4dc2c27 
…ustment method
@s3ich4n
docs(chapter09): add notes on assertions and variable splitting for c…
36340c4 
…larity
@s3ich4n
feat(chapter09): add Person and TelephoneNumber classes with basic fu…
9fc1fdd 
…nctionality and tests
@s3ich4n
refactor(chapter09): update TelephoneNumber and Person classes
4e7da78 
- for immutability and equality checks
@s3ich4n
feat(chapter09): implement Customer and Order classes with basic func…
eaaaf12 
…tionality and tests
@s3ich4n
feat(chapter05): add CustomerRepository class and legacy singleton fu…
9532082 
…nctions for customer management
@s3ich4n
feat(chapter09): add README for magic literal refactoring guidelines
3d86e71
@s3ich4n
docs(chapter09): marked chapter 9 as complete
3884580
@s3ich4n s3ich4n added documentation Improvements or additions to documentation refactor Refactored labels Mar 17, 2025
@s3ich4n s3ich4n self-assigned this Mar 17, 2025
@s3ich4n
Copy link
Owner Author

s3ich4n commented Mar 17, 2025

📝 Test & Coverage Report

Metric Value
✅ Passed 104
❌ Failed 0
⏩ Skipped 0
⏱️ Time 0.11s
📊 Line Coverage 96.80%

@s3ich4n s3ich4n merged commit b8d0189 into main Mar 17, 2025
1 check passed
@s3ich4n s3ich4n deleted the chapter09 branch March 26, 2025 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@s3ich4n s3ich4n

Labels

documentation Improvements or additions to documentation refactor Refactored

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[FEATURE] 리팩터링 챕터 9

1 participant

@s3ich4n