Feat/더하기 기능 추가

[127lucas]

주요 작업 내용

• 덧셈 기능 추가

• 기호 말고 잘못된걸 입력할시 오류라고 뜨는 기능 추가

관련이슈

Closes #9

[dohun0310]

안녕하세요, 동규님!

이번 리뷰는 코드의 정답 여부를 평가하기보다는,
Git과 GitHub를 어떻게 활용하셨는지, 그리고 각 선택에 어떤 의도가 있었는지를 함께 살펴보는 데 중점을 두고 있습니다.

이번 과제를 통해 Git과 GitHub에 한층 더 익숙해지셨기를 바랍니다 :)

[dohun0310]

Issue 제목을 보면 [feet] 덧셈 기능 추가라고 작성되어 있습니다.

혹시 [Feat]를 의도하신 게 맞을까요?

작은 오타이지만, Issue 제목은 팀원 모두가 확인하는 공간입니다.
나중에 히스토리를 추적할 때도 활용되기 때문에 정확하게 작성하는 게 중요해요.

만약 Issue 제목에 오타가 있다는 걸 나중에 발견했다면 어떻게 하는 게 좋을지, 한번 생각해보셨나요? Issue를 수정하는 방법과, 닫고 새로 여는 방법 중 어떤 선택이 더 적절한지도 탐구해보시면 어떨까요?

[dohun0310]

Issue를 보면 작업 항목들이 체크박스로 나열되어 있는데,
PR에서 작업을 완료하셨음에도 불구하고 체크박스가 하나도 체크되지 않은 상태인 걸 확인했어요.

체크박스를 Issue에 작성하신 이유가 있으셨나요?
그리고 이 체크박스가 어떤 용도로 활용되는 건지에 대해서도 한번 생각해보셨으면 해요.

Issue의 체크박스는 단순한 메모가 아니라,
해당 작업이 완료되었는지를 팀원들이 한눈에 파악할 수 있도록 돕는 역할을 해요.
PR이 올라간 시점이나 작업이 완료된 시점에 체크박스를 업데이트하지 않으면,
나중에 이 Issue를 보는 팀원 입장에서는 작업이 아직 진행 중인지 완료된 건지 알 수가 없게 되거든요.

앞으로 Issue를 작성할 때 체크박스를 어떻게 활용하면 좋을지 한번 생각해보시면 좋겠어요 :)

[dohun0310]

PR 히스토리를 보니 처음에는 main 브랜치를 대상으로 PR을 올리셨다가,
이후에 team-1으로 변경하신 것을 확인했어요.

처음에 main을 대상으로 선택하신 이유가 있으셨나요?
그리고 team-1으로 바꾸신 건 어떤 이유에서였는지도 궁금합니다.

PR을 올릴 때 base 브랜치를 어디로 설정해야 하는지는 팀의 브랜치 전략과 직접 연결되는 부분이에요.
이번 과제에서 팀이 정한 브랜치 전략을 기준으로,
본인의 작업이 최종적으로 어디에 합쳐져야 하는지 어떻게 판단하셨는지 이야기해주세요.

[dohun0310]

Labels와 Type 설정에 대해서도 여쭤보고 싶어요.

Issue에 feature Label과 Feature Type이 지정되어 있는 걸 확인했는데요.

이 두 가지를 각각 어떤 용도로 사용하는 건지 알고 계신가요?
언뜻 보면 비슷해 보이지만, Labels와 Type은 GitHub에서 서로 다른 목적으로 설계된 기능이에요.
이 둘이 어떻게 다른지 한번 찾아보시면 좋을 것 같아요.

또, 이 설정들이 팀원들과 충분한 논의를 거쳐서 정해진 건지도 궁금합니다.
예를 들어 "어떤 작업에 어떤 Label을 붙일지", "Type은 어떤 기준으로 선택할지" 같은 부분을 팀 내에서 미리 합의해두지 않으면,
나중에 Issue가 쌓였을 때 필터링이나 분류가 제대로 되지 않아 오히려 혼란스러워질 수 있거든요.

이번에 이 설정들을 지정하신 기준이 어디서 나온 건지 이야기해주시면 좋겠습니다.

[dohun0310]

PR 설명에 "임포트 기능"이 작업 내용으로 명시되어 있는데,
실제 코드에는 import가 없는 것을 확인했어요.

PR 설명은 실제로 작업한 내용을 기준으로 작성되어야 하는데,
이 부분은 어떻게 된 건지 여쭤봐도 될까요?

[127lucas]

아! import 랑 input이랑 햇갈려서 잘못 적은것 같습니다

[127lucas]

Q1." 처음에 main을 대상으로 선택하신 이유가 있으셨나요?
그리고 team-1으로 바꾸신 건 어떤 이유에서였는지도 궁금합니다.
PR을 올릴 때 base 브랜치를 어디로 설정해야 하는지는 팀의 브랜치 전략과 직접 연결되는 부분이에요.
이번 과제에서 팀이 정한 브랜치 전략을 기준으로,
본인의 작업이 최종적으로 어디에 합쳐져야 하는지 어떻게 판단하셨는지 이야기해주세요."

A1. 처음 PR을 생성할 때 기본 설정이 main으로 되어 있었고 team1으로 바꿨는데.. PR에 대해 알아볼게 있어 검색후 다시 돌아와 보니 새로고침 되어 있어서 다시 main으로 설정이 되어 있더라구여.. 그걸 못보고 바로 보내서 main으로 보내버렸습니다
제 판단으로는 제가 맡은 덧셈 기능은 팀 프로젝트의 일부분이므로 팀원들과의 코드 충돌을 확인하고 결과물을 만들기 위해 우리 팀의 공통 작업 공간인 team-1 브랜치를 base로 설정하는 것이 더 적절해 보입니당.

[127lucas]

Q2. Labels와 Type 설정에 대해서도 여쭤보고 싶어요.
Issue에 feature Label과 Feature Type이 지정되어 있는 걸 확인했는데요.
이 두 가지를 각각 어떤 용도로 사용하는 건지 알고 계신가요?
언뜻 보면 비슷해 보이지만, Labels와 Type은 GitHub에서 서로 다른 목적으로 설계된 기능이에요.
이 둘이 어떻게 다른지 한번 찾아보시면 좋을 것 같아요.
또, 이 설정들이 팀원들과 충분한 논의를 거쳐서 정해진 건지도 궁금합니다.
예를 들어 "어떤 작업에 어떤 Label을 붙일지", "Type은 어떤 기준으로 선택할지" 같은 부분을 팀 내에서 미리 합의해두지 않으면,
나중에 Issue가 쌓였을 때 필터링이나 분류가 제대로 되지 않아 오히려 혼란스러워질 수 있거든요.
이번에 이 설정들을 지정하신 기준이 어디서 나온 건지 이야기해주시면 좋겠습니다.

A2. Label 과 Type 의 차이는 모르고 있었지만 지금 검색해본 결과
Label 은 검색하기 쉽게 다각도로 보여주는 태그 시스템과 비슷한것 같고
Type는 대분류 즉 그 issue에 대한 정체성을 보여주고 있는것 같습니다.
또 이번 계산기 프로젝트는 대부분 새로운 기능을 만드는 작업(Feature) 위주이다 보니
노션에 정의된 많은 태그 중에서도 feat 위주로만 사용하게 되었습니다.
그러다 보니 자연스럽게 Label과 Type도 feature 외에는 다양하게 활용하지 못했던 것 같네요.
팀원들과 기본적인 논의는 거쳤지만
말씀하신 대로 이슈가 쌓였을 때의 혼란을 방지하기 위해 어떤 상황에
어떤 Label과 Type을 적용할지에 대해서는 추가적인 논의가 필요할것 같습니다.