Feat/srp#12
Feat/srp #12
Uh oh!
There was an error while loading. Please reload this page.
Conversation
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
README.md Outdated
| - Retrieve the version number of the Python package | ||
| - Render itself as an HTML element | ||
| - 파이썬 패키지의 버전 번호를 검색 | ||
| - 그 자체를 HTML 요소로써 렌더링 |
There was a problem hiding this comment.
itself의 지칭은 VersionCommentElement 클래스가 될 것이나, 자칫하면 원문의 의도를 왜곡할 것 같아 그 자체라는 직역안을 반영했습니다.클래스 로 변경하는 게 나아 보이는지 다른 분들의 의견이 궁금합니다. 🤔
There was a problem hiding this comment.
요부분은 좀 더 논의가 필요할 것 같아요!!
뭔가 원문이 무슨 말을 하고 싶은지는 알 것 같은데, 한국어로 표현하자니 뭔가 애매한 그런 느낌...?
한번 다음에 같이 이야기해보아요!
There was a problem hiding this comment.
저도 그 자체라는 직역은 처음보는 사람 입장에서 헷갈릴 여지가 있다고 생각해요.
문장의 의도는 bad case의 경우, 2가지 책임이 서로 dependent 하다는 걸 보여주고 싶은 것 같은데,
itself가 클래스의 멤버함수인 def get_version(self) -> str: 라는 걸 명확히 하면 좋을 것 같아요.
README.md Outdated
| ### **단일 책임 원칙(Single Responsibility Principle (SRP))** | ||
| Robert C. Martin writes: | ||
| Robert C. Martin 는 다음과 같이 저술한 적이 있습니다. |
There was a problem hiding this comment.
Robert C. Martin 는 다음과 같이 저술하였습니다.
로 축약해 보아도 괜찮을 것 같아욤! 😊
kyc3492 commented Jun 18, 2023
이 PR은 6.14. 야근세션 부로 해결된 PR입니다. 확인 이모지 남겨주시면 제가 병합 하겠습니다~~ |
SRP 부분 번역을 완료하여 PR 올립니다.
진행하면서 의문이 있었던 부분은 코멘트 남기겠습니다.
추가 개선사항이나 의문점이 있다면 자유롭게 코멘트 부탁드립니다! 😁