Conversation
kyc3492
commented
Jun 8, 2023
kyc3492
commented
Jun 8, 2023
kyc3492
commented
Jun 8, 2023
kyc3492
commented
Jun 8, 2023
README.md
Outdated
| - Retrieve the version number of the Python package | ||
| - Render itself as an HTML element | ||
| - 파이썬 패키지의 버전 번호를 검색 | ||
| - 그 자체를 HTML 요소로써 렌더링 |
Author
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: 라는 걸 명확히 하면 좋을 것 같아요.
FacerAin
reviewed
Jun 12, 2023
README.md
Outdated
| ### **단일 책임 원칙(Single Responsibility Principle (SRP))** | ||
|
|
||
| Robert C. Martin writes: | ||
| Robert C. Martin 는 다음과 같이 저술한 적이 있습니다. |
There was a problem hiding this comment.
Robert C. Martin 는 다음과 같이 저술하였습니다.
로 축약해 보아도 괜찮을 것 같아욤! 😊
Author
|
이 PR은 6.14. 야근세션 부로 해결된 PR입니다. 확인 이모지 남겨주시면 제가 병합 하겠습니다~~ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
SRP 부분 번역을 완료하여 PR 올립니다.
진행하면서 의문이 있었던 부분은 코멘트 남기겠습니다.
추가 개선사항이나 의문점이 있다면 자유롭게 코멘트 부탁드립니다! 😁