Merged
Conversation
chore: init and install cypress module
Merge byhhh2/javascript-calculator
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.
처음 한 페어프로그래밍이라 너무 떨리고 재밌었던 것 같아요!
코드를 다른 관점에서 볼 수 있다는 점이 가장 좋았던 것 같네요.
특히 제가 발견 못한 오류를 페어님이 발견하시고 페어님이 발견 못한 오류를 제가 발견할 때 페어프로그래밍의 가장 큰 장점을 볼 수 있었던 것 같습니다.
말을 하면서 코딩한다는 점에서 조금 집중이 덜 되긴 했지만, 그럴 때에는 잠시 각자 생각하는 시간을 갖고 다시 모였습니다. 😄
처음으로 테스트 코드도 작성해보았는데 생각보다 어렵지 않아서 예전에도 도전해볼껄 후회가 되네요. 꾸준히 cypress를 사용할 것 같아요!