search

dongju-chae / nns-contributhon

4 stars 1 forks source link

[Fix] I fixed my commit. #4

Closed suehdn closed 4 years ago

suehdn commented 4 years ago

I tried hard to fix it.. Thank you.

Signed-off-by: Hyesu Ahn linim@naver.com

dongju-chae commented 4 years ago

그리고 commit 생성 이후, git commit --amend 을 통해서 현재 commit에 대한 간단한 설명을 넣어주셔야 합니다.

dongju-chae commented 4 years ago

@suehdn 다 좋은데 commit 텍스트 메시지만 고치시면 될 것 같아요~ 그리고 pull request의 텍스트 또한 commit에 대한 설명으로 고쳐야되요 ㅎㅎ git commit --amend 한 이후 나오는 에디터 창에서

[Keyword] 제목 ...

내용 ...

signed-off ...

이런식으로 signed-off가 가장 마지막으로 가야 됩니다. 다시 force-push 해주시면 됩니다 ㅎㅎ

suehdn commented 4 years ago

@dongju-chae 아 죄송합니다ㅡㅠㅠ 제가 작성하고 나서 댓글을 봤어요.. 다시 해보겠습니다

dongju-chae commented 4 years ago

@suehdn 아니에요 ㅎㅎ 거의 다 왔습니다.

suehdn commented 4 years ago

아 마지막 잊었네요 다시할게요

dongju-chae commented 4 years ago

@suehdn commit 내용중에 rebase를 하다보면 제목이 comment 안에 들어가는데 (지금 update, re-update 처럼..) 이거 그냥 지워주셔도 되요. rebase 된 commit의 제목들이기 때문에 필요가 없어요.

dongju-chae commented 4 years ago

@suehdn 잘 하셨는데요 ㅎㅎ 지금 add my account/final update 이거랑 [Git,PR]Git 사용법 및 PR 올리기 이게 둘다 제목 처럼 보이거든요. 한글로 commit을 만드실 꺼면 add ~ 이거 그냥 지우고 [Git, PR] 이거를 제일 위로 오게 하면 될 것 같아요. (근데 commit은 되도록 한글 없이 영어로만 하는게 좋습니다 ^^ 최대한 간단한 영어로)

이 페이지 제일 상단에 Pull request의 메시지도 웹페이지 상에서 edit 할수가 있습니다.

suehdn commented 4 years ago

이정도로 commit을 하면 될까요??

dongju-chae commented 4 years ago

ㅎㅎ 네 잘 하셨습니다. 근데 딱 두가지만 더 고쳐 주세요 ㅋㅋㅋㅋ 죄송해요 ㅠㅠ

  1. Commit 메시지 제목에서 [ ] 이후에 간단한 제목을 더 적어주세요..
  2. Pull Request의 내용에서는 commit의 제목 ([ ] ...)을 포함할 필요가 없습니다. 이거는 오히려 pull request의 제목으로 하는게 맞습니다.
suehdn commented 4 years ago

감사합니다!!!

dongju-chae commented 4 years ago

@suehdn ㅋㅋ 고생 많으셨어요. 나중에 nnstreamer 도 이런식으로 commit 만들고 PR 올려주시면 됩니다.

suehdn commented 4 years ago

@dongju-chae 잊지 않고 꼭 이 방식으로 하겠습니다.. 감사합니다 ㅎㅎ!!

dongju-chae commented 4 years ago

@suehdn 에고 ㅜㅜ 올리신 패치를 제가 제대로 못 봤었어요. 올리신 패치에 diff 내용이 들어가 있었네요.. 제가 해당 부분 수정한 것을 참고하세요~ https://github.com/dongju-chae/nns-contributhon/pull/5

추가로 설명드리자면, 그 이슈에 적힌 git diff는 파일의 변경 사항을 출력해주는 것으로서, 그 결과를 통째로 넣으라는 뜻이 아니고, ID를 추가하는 등의 파일 수정을 한 이후, git diff를 하면 저런식으로 출력이 되고 자신이 수정한 파일들의 변경사항을 비교해서 다시 확인해 볼 수 있다는 의미 입니다 (-는 없어지는 line, +는 새로 추가되는 line).

suehdn commented 4 years ago

@dongju-chae 제가 처음에 했을때는 그 부분을 안넣었는데 git diff했을때 저 부분을 제가 확인을 못했는지 계속 넣고 진행했었네요... 수정해주셔서 감사합니다ㅎㅎ