코드 주고받기 마일스톤 지정

Skm1221 commented 9 years ago

코드 주고 받기 마일스톤 지정을 해봤습니다.

먼저 간단하게 코드주고받기에 마일스톤을 지정하기위해 view에서 마일스톤 지정을 하는 탭을 추가 시켰고 model에서 pullRequest에 마일스톤이라는 속성을 추가시키고 마일스톤에 역시 pullRequest라는 set을 추가시켰습니다.

2번째로 create 한다음 수정을 통해 milestone을 변경역시 가능하도록 pullRequestApp.java파일에서 생성하는 부분과 수정하는 부분에 setMilestone라는 함수를 추가시켜 마일스톤을 지정 및 변경이 가능하도록 했습니다.

그후 마일스톤 창에서 issue뿐만아니라 pullrequest도 볼 수 있도록 구현했습니다.

먼저 우선적으로 마일스톤의 status에 보면 issue만 가지고 열림과 닫힘수를 고려하여 completionrate를 보여주는데 여기에 pullRequest것도 추가시켰습니다.

그다음 마일스톤을 보면 맨 처음에 issue의 것이 나오고 그다음에는 탭에 현재 보고있는 마일스톤을 지정한 pullRequest의 리스트를 보여주게 했습니다.

여기서 pullRequest의 닫힘에는 닫힘과 병합 모두 포함시켰습니다.

현재 구현하지 못한부분은 issue에서는 수정 화면으로 가지 않는 상태에서 탭에서 마일스톤을 변경하는것만으로도 마일스톤의 변경이 가능한데 그부분을 구현하지 못했고 pullRequest를 클릭했을시 보여주는 화면에서 milestone을 보여주는 탭이 원래의 format으로 나오지 못하는데 이 부분역시 이유를 모르겟습니다.

마지막으로 milestone에서 이슈의 리스트를 보여주는 탭과 pullRequest를 보여주는 탭 2개가 있는데 예를 들어 issue들의 리스트를 보여주는 탭에서 열림 혹은 닫힘을 클릭할때 아래있는 pullRequest들의 리스트를 보여주는 탭 역시 issue 탭에서 클릭했던 열림 혹은 닫힘으로 탭의 herf가 바뀌게 됩니다.

keesun commented 9 years ago

커밋은 squash 해서 (git rebase -i HEAD~n) 작업 단위별로 합쳐주시고요 커밋 메시지를 정성껏 작성해 주세요.

Skm1221 commented 9 years ago

pullRequest를 다시 넣으면 되는거죠 ??

keesun commented 9 years ago

로컬에서 rebase 사용해서 squash 하셨다면 코드를 보낸 브랜치에 강제 푸쉬 하시면 이 풀리퀘가 갱신되니까 다시 보낼 필요는 없습니다.

— Keesun Baik백기선

On Tue, Jun 23, 2015 at 8:49 PM, Skm1221 notifications@github.com wrote:

pullRequest를 다시 넣으면 되는거죠 ??

Reply to this email directly or view it on GitHub: https://github.com/naver/yobi/pull/933#issuecomment-114465200

Skm1221 commented 9 years ago

음.. 우선 저기서 HEAD~5까지는 합쳤는데 그다음을 나머지끼리를 헤드랑 합치면 단위별로 합쳐지나요 ?????

아 그리고 강제푸쉬는 어떻게 하나요 ? git push origin test+했는데 error: src refspec test+ does not match any. error: failed to push some refs to 'https://github.com/Skm1221/yobi.git'

이렇게 되던데 ..ㅠㅠ

whiteship commented 9 years ago

어떻게 합치시려는 건지 잘 모르겠지만.. 강제푸쉬는 git push -f 옵션을 사용하세요

— Keesun Baik백기선

On Tue, Jun 23, 2015 at 10:31 PM, Skm1221 notifications@github.com wrote:

음.. 우선 저기서 HEAD~5까지는 합쳤는데 그다음을 나머지끼리를 헤드랑 합치면 단위별로 합쳐지나요 ????? 아 그리고 강제푸쉬는 어떻게 하나요 ? git push origin test+했는데 error: src refspec test+ does not match any. error: failed to push some refs to 'https://github.com/Skm1221/yobi.git'

이렇게 되던데 ..ㅠㅠ

Reply to this email directly or view it on GitHub: https://github.com/naver/yobi/pull/933#issuecomment-114503150

Skm1221 commented 9 years ago

음... git config --global push.default matching 한 후 git push -f를 진행하였는데 왜 안바뀔까요 ...

Skm1221 commented 9 years ago

오 변경 했어요!!!!! 확인 부탁드립니다

Skm1221 commented 9 years ago

@keesun 확인 부탁드립니다!

naver / yobi

코드 주고받기 마일스톤 지정 #933

pullRequest를 다시 넣으면 되는거죠 ??

이렇게 되던데 ..ㅠㅠ