Skip to content

Conversation

@kyc3492
Copy link

@kyc3492 kyc3492 commented Jun 8, 2023

SRP 부분 번역을 완료하여 PR 올립니다.
진행하면서 의문이 있었던 부분은 코멘트 남기겠습니다.
추가 개선사항이나 의문점이 있다면 자유롭게 코멘트 부탁드립니다! 😁

README.md Outdated
- Retrieve the version number of the Python package
- Render itself as an HTML element
- 파이썬 패키지의 버전 번호를 검색
- 자체를 HTML 요소로써 렌더링
Copy link
Author

Choose a reason for hiding this comment

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

itself의 지칭은 VersionCommentElement 클래스가 될 것이나, 자칫하면 원문의 의도를 왜곡할 것 같아 그 자체라는 직역안을 반영했습니다.
클래스 로 변경하는 게 나아 보이는지 다른 분들의 의견이 궁금합니다. 🤔

Choose a reason for hiding this comment

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

요부분은 좀 더 논의가 필요할 것 같아요!!
뭔가 원문이 무슨 말을 하고 싶은지는 알 것 같은데, 한국어로 표현하자니 뭔가 애매한 그런 느낌...?
한번 다음에 같이 이야기해보아요!

Choose a reason for hiding this comment

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

저도 그 자체라는 직역은 처음보는 사람 입장에서 헷갈릴 여지가 있다고 생각해요.
문장의 의도는 bad case의 경우, 2가지 책임이 서로 dependent 하다는 걸 보여주고 싶은 것 같은데,
itself가 클래스의 멤버함수인 def get_version(self) -> str: 라는 걸 명확히 하면 좋을 것 같아요.

@kyc3492 kyc3492 self-assigned this Jun 8, 2023
@kyc3492 kyc3492 requested review from FacerAin and jinmyeongAN June 8, 2023 08:14
README.md Outdated
### **단일 책임 원칙(Single Responsibility Principle (SRP))**

Robert C. Martin writes:
Robert C. Martin 다음과 같이 저술한 적이 있습니다.

Choose a reason for hiding this comment

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

Robert C. Martin 는 다음과 같이 저술하였습니다.
로 축약해 보아도 괜찮을 것 같아욤! 😊

@kyc3492
Copy link
Author

kyc3492 commented Jun 18, 2023

이 PR은 6.14. 야근세션 부로 해결된 PR입니다. 확인 이모지 남겨주시면 제가 병합 하겠습니다~~

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.

4 participants