Skip to content
This repository has been archived by the owner on Sep 12, 2018. It is now read-only.

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

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

wants to merge 3 commits into from

Conversation

Skm1221
Copy link

@Skm1221 Skm1221 commented Jun 23, 2015

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

먼저 간단하게 코드주고받기에 마일스톤을 지정하기위해 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
Copy link
Contributor

keesun commented Jun 23, 2015

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

@Skm1221
Copy link
Author

Skm1221 commented Jun 23, 2015

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

@keesun
Copy link
Contributor

keesun commented Jun 23, 2015

로컬에서 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:
#933 (comment)

@Skm1221
Copy link
Author

Skm1221 commented Jun 23, 2015

음.. 우선 저기서 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
Copy link

어떻게 합치시려는 건지 잘 모르겠지만.. 강제푸쉬는 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:
#933 (comment)

@Skm1221
Copy link
Author

Skm1221 commented Jun 24, 2015

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

Change create and view of html of git to add milestone setting menu,
	   Then make the function to count the pullrequests of specific milestone, but it makes error Because i don't make attribute to each file(pullrequest of milestone, miletone of pullrequest) so i did annotation that function
…llRequest, then Add each attribute at DataBase

Change the git edit view to add milestone setting menu.
In order to count the number of pullrequest of specific Milestone, Make
count function in milestone.Java file.
make category of All state
modify createSearchExpressionList to distinguish pullrequest's milestone
modify status of milestone. reflect the number of pullRequest to completion rate of milestone.
@Skm1221
Copy link
Author

Skm1221 commented Jun 25, 2015

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

@Skm1221
Copy link
Author

Skm1221 commented Jun 25, 2015

@keesun 확인 부탁드립니다!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants