Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Caffe2 Pull Request 현황 #10

Open
haracejacob opened this issue Jun 1, 2017 · 5 comments
Open

Caffe2 Pull Request 현황 #10

haracejacob opened this issue Jun 1, 2017 · 5 comments

Comments

@haracejacob
Copy link
Contributor

facebookarchive/caffe2#719

소스코드 수정해서 Pull-Request 했습니다!

답변오는대로 issues 업데이트 하겠습니다.

@haracejacob
Copy link
Contributor Author

Facebook의 경우 Contributor License Agreement(CLA)를 해야합니다.
Pull-Request시 유저의 CLA가 없으면 싸인을 해달라는 글이 남습니다. 그리고 링크를 타고 싸인을 하면 자동적으로 CLA를 했다고 뜹니다.

@ASTONISHER
Copy link
Contributor

오오오오 됐어요! 와와!! 짱이다!!!

@haracejacob
Copy link
Contributor Author

아직 된건 아닌거같아요
~ has imported this pull request. 라는게 뭘 의미하는건지 모르겠네요 ㅠㅠ
merge 했으면 close 하는거 아닌감...?

@HWhale
Copy link
Contributor

HWhale commented Jun 11, 2017

facebookarchive/caffe2#780
마찬가지로 C++ tutorial에 대한 Pull-request 완료했습니다.
Contribution 문서 참조해서 navigation bar까지 바뀌도록 했고, 위에서 언급한 CLA도 동의처리 하였습니다.

할 수 있는 부분은 다 한 것 같고.. 피드백이 올 때까지 기다리면 될 것 같습니다.

@haracejacob
Copy link
Contributor Author

facebookarchive/caffe2@4c76b74

제 Pull-request가 커밋되었습니다!

짝짝짝

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

No branches or pull requests

3 participants