-
Notifications
You must be signed in to change notification settings - Fork 0
[Docs] README를 업데이트합니다. #313
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.
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
Conversation
YunDaeHyeon
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
📄 PR Code Review Report
🔍 Summary
- 현재 변경사항은 프로젝트 문서에 국한되어 있으며, 애플리케이션의 런타임 동작이나 안정성에 직접적인 영향을 미치지 않습니다. 주로 프로젝트 경험 목록을 업데이트하고 새로운 링크를 추가하는 내용입니다.
⚠️ Key Issues
- 애플리케이션의 기능, 성능 또는 안정성에 영향을 줄 수 있는 중대한 코드상의 문제는 발견되지 않았습니다. 문서 내용 자체에 치명적인 오류나 오해의 소지가 있는 부분은 없습니다.
🛠 Improvement Suggestions
- 기술적인 코드 리뷰 관점에서 이 문서 변경사항에 대한 개선 제안은 없습니다.
✅ Positive Observations
- 프로젝트의 경험과 성과를 상세히 설명하는 새로운 항목들이 추가되어 문서의 내용이 더욱 풍부해졌습니다. 새로 추가되거나 수정된 링크들도 적절하게 구성되어 있어 정보 접근성을 높였습니다. 전반적으로 문서를 최신 상태로 유지하려는 노력이 긍정적입니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- 모니터가 fhd라 그런건지 모르겠지만, 각 배지들의 텍스트가 잘 안보여요. 가능하다면 볼드체가 어떤지..
- 배지 모양 애플스럽게 flat 어떨까요
- Combine 쓰는 곳이 사실상 CaptureButton의 디바운싱말고 쓰는 곳이 없는데, 차라리 저희가 열심히 리팩토링한 AsyncStream이 어떤지... async await도 AsyncStream에 포함될듯 합니다 (사진은 import Combine 주석처리)
- 멀티피어, 워치 Connectivity 사이에 띄어쓰기 추가하는건 어떨까요. 공식 문서도 띄워둔 상태라... (물론 url은 띄어쓰기가 없긴 합니다)
sangYuLv
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다 👍
GRJeon
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
위키 마스터 👍
4b8250e to
b413d43
Compare
📝 작업 내용
📌 요약
README의
핵심 경험을 최신화했습니다.추가적인 README 수정 요소가 있다면 말씀해주세요!
바로 반영하겠습니다.
README
위 링크에서 README를 확인해주세요!