Conversation
LEEMINGU03
requested review from
Donghwan814,
dohun0310,
leesj0706 and
seojoon1
and removed request for
1028ragon,
127lucas,
LEEMINGU03 and
ohsanghy09
|
정훈님! |
| def subtract(a, b): | ||
| try: | ||
| return(float(a) - float(b)) |
There was a problem hiding this comment.
데이터 타입을 실수형으로 설정하신 건 바람직한 부분입니다.
다만 다른 팀원들과 데이터 타입에 대한 차이가 발생했는데, 팀원분들과 충분히 협의가 된 내용인지 궁금합니다!
이러한 질문을 드리는 이유는 이번 과제가 협업관련 과제기 때문에 충분한 소통이 되었는지 확인하기 위해 질문드렸습니다.
There was a problem hiding this comment.
팀과 같이 코드를 작성하고 설계하는 것 보다 혼자서 Github와 Git의 사용 방법에 더 집중하느라 코드 쪽을 신경쓰지 못했던 것 같습니다. 앞으로는 팀원들과 충분히 소통하여 협업하는 것에도 신경을 더 써야겠다고 생각하게 되었습니다.
There was a problem hiding this comment.
팀과 같이 코드를 작성하고 설계하는 것 보다 혼자서 Github와 Git의 사용 방법에 더 집중하느라 코드 쪽을 신경쓰지 못했던 것 같습니다. 앞으로는 팀원들과 충분히 소통하여 협업하는 것에도 신경을 더 써야겠다고 생각하게 되었습니다.
넵! 좋은 것 같습니당
| except ValueError: | ||
| return "숫자를 입력하십시오." | ||
| # 뺄셈 구현 No newline at end of file |
|
일단 먼저 commit message 관련해서 질문 드립니다. 노션 team1 페이지에서 커밋 메세지 방식을 보면 태그명/내용 형식으로 작성이라고 적혀있는데, 그래서 팀원들과 충분한 의사소통이 있었는지, 아니면 팀원들이 정해놓은 규칙을 확인을 못하신건가요? 만약 확인을 하셨다면 커밋 메세지가 다른 이유가 궁금합니다. |
|
그 다음은 pr컨벤션 관련해서 질문 드립니다. 노션 페이지에는 pr컨벤션이 따로 작성되어있지 않은걸로 확인되었습니다. 하지만 다른 팀원분들은 pr컨벤션이 통일된걸로 확인되고, 정훈님의 pr컨벤션은 다른 것 같아서 혹시 팀원들께 전달을 못받은건지, 아니면 놓치신 부분인지 궁금합니다. 만약 놓치신 부분이라면 놓치신 이유도 있는지 궁금합니다. |
|
마지막으로 branch와 history 관련해서 질문드립니다. 일단 branch 관련해서 질문드립니다. 지금 정훈님이 만드신 브랜치는 revert, Feat/sub#10, Feat/subtract#10 총 3개로 확인됩니다. 뺄셈 브랜치를 왜 두개나 만드신지 궁금하고, revert 브랜치를 만드신 계기가 궁금합니다. 다음으로는 history 관련해서 질문드립니다. 정훈님의 history를 보면 force push를 2번 사용하신걸로 확인했는데, 왜 2번이나 force push를 사용하신건지 궁금합니다. |
로컬 저장소와 Github의 저장소를 연동하는데 어려움을 겪어 노션의 팀에서 정해진 방식을 확인하는 것에 신경쓰지 못한 것 같습니다. 앞으로는 팀에서 합의된 규칙을 지키는 것에 더욱 신경쓰도록 하겠습니다. |
팀과의 협업 보다 혼자서 로컬 저장소와 Github 저장소의 연동 문제에 더 집중하느라 Notion의 확인을 놓쳤던 것 같습니다. 앞으로는 좀 더 꼼꼼히 확인하도록 하겠습니다. |
revert 브랜치의 경우 실수로 team-1에 Feat/subtract#10 브랜치를 Pull Requests를 시도하여 요청만 하여야 하는데 Merge까지 해버려서 team-1 브랜치를 Merge 전으로 되돌리기 위해서 방법을 찾다가 시도해 본 결과입니다. Feat/sub#10 브랜치와 Feat/subtract#10 두 가지가 있는 이유는 위에서 team-1 브랜치를 Merge 전으로 되돌리려고 한 후 다시 Pull Requests를 요청하려고 시도하였는데 이미 Merge 되어 있어서 안된다고 오류가 발생하자 한 팀원 분께서 아얘 다른 브랜치로 시도해보라고 하셔서 브랜치가 2개가 되게 되었습니다. force push를 2번 사용한 이유에 대해서는 이미 force push가 적용이 되었는지 확인하지 못하여 2번 시도하게 된 것 같습니다. 앞으로는 현 상황에 대해 잘 확인하도록 하겠습니다. |
|
|
6 participants
주요 작업 내용
뺄셈 기능의 추가
문자열 입력 시 오류 발생
관련 이슈
Closes #17