Conversation
This file contains 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
metacode22
requested review from
hyesungoh,
100Gyeon,
Dongkyuuuu,
gutenLEE,
Orchemi,
sangbooom,
Seojunhwan,
sjoleee and
sumi-0011
as code owners
hyesungoh
approved these changes
Sep 10, 2024
Comment on lines
+21
to
+23
| 학습 및 기술 갈고 닦기. 많은 기술이 팀 전체로 퍼졌을 때 더 효과적이다. 팀원들을 전도할 계획을 세워라. | ||
|
|
||
| > 사내 스터디 2개를 운영하고 있는데, 개발 관련 팁을 알려주는 슬랙 봇을 만드는 등 더 열심히 전도해봐야겠어요. |
There was a problem hiding this comment.
React 공식 문서 스터디, 한 주간 겪은 프론트 지식이나 이슈 공유하는 스터디 이렇게 2개 해보고 있어요!
Comment on lines
+27
to
+29
| 팀이 하나로 의사소통하게 도와주는 비결이 있는데, 그것은 프로젝트에 유별난 이름을 지어주는 것이다. | ||
|
|
||
| > toss/slash를 모방해서 회사에서 union이라는 패키지 모노레포를 만들고 있는데, 더 유별난 걸 해볼걸 그랬네요. |
Comment on lines
+35
to
+37
| "진짜 상황 테스트"를 목표로 하는 것이 중요하다. 즉, 테스트 환경은 실제 환경과 최대한 비슷해야 한다. 두 환경의 차이에서 버그가 번식한다. | ||
|
|
||
| > e2e 환경을 구축했던 것이 생각나는데요, api를 mocking하는 것이 아니라 테스트 서버 api를 사용했었어요. 책에서 말한 것처럼 최대한 실제 환경과 비슷하게 하기 위함이었는데 백엔드에 의존성이 생긴다는 점이 단점이 아쉬웠어요. 근데 백엔드에 자그마한 변경사항이 생겼을 때, 이로 인해 e2e가 실패한다면 곧바로 프론트에서도 변경사항을 캐치할 수 있겠다는 생각이 들었어요. |
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.
덕분에 개발 책 하나 정독해보네요😀 다들 고생하셨습니다!