kakao / khaiii

Kakao Hangul Analyzer III
Apache License 2.0
1.41k stars 284 forks source link

도커 이미지 빌드 및 동작 가능한 상태로 Dockerfile 수정 #98

Closed tooget closed 3 years ago

tooget commented 3 years ago

설명 (Description)

도커 이미지 빌드 및 실행이 되지 않아 수정합니다.

카이 형태소 분리 테스트 입니다. 카이 카이/NNP 형태소 형태소/NNG 분리 분리/NNG 테스트 테스트/NNG 입니다. 이/VCP + ㅂ니다/EF + ./SF



~~_겁내지 말아요, 저희는 한글을 사랑합니다._~~

개발자를 위한 가이드 (Developer's Guide)
----
만약 khaiii에 pull request가 처음이라면 [개발자를 위한 가이드](https://github.com/kakao/khaiii/wiki#%EA%B0%9C%EB%B0%9C%EC%9E%90%EB%A5%BC-%EC%9C%84%ED%95%9C-%EA%B0%80%EC%9D%B4%EB%93%9C) 문서들을 한번 읽어보시길 권고드립니다.

If this is your first pull request for khaiii, please see the [Developer's Guide](https://github.com/kakao/khaiii/wiki#%EA%B0%9C%EB%B0%9C%EC%9E%90%EB%A5%BC-%EC%9C%84%ED%95%9C-%EA%B0%80%EC%9D%B4%EB%93%9C).

체크 리스트 (Checklist)
----
pull request 전에 아래 체크 리스트들을 만족하는 지 확인한 후 체크('x') 표시를 해주시기 바랍니다.

Before you submit pull requests, please check(set 'x') to the checklist below.

- [x] master 브랜치가 아니라 **develop** 브랜치에 머지하도록 pull request를 작성 중이신가요? (Did you merge into **develop** branch not master?)
- [ ] `build/test/khaiii` 프로그램을 실행하여 **테스트**가 성공했나요? (Did all **tests** are passed when you ran as `build/test/khaiii`)
- [ ] **PyLint** 툴을 실행하여 발생한 에러를 모두 수정하셨나요? (Did you fix all errors after running **PyLint**?)
- [ ] **CppLint** 툴을 실행하여 발생한 에러를 모두 수정하셨나요? (Did you fix all errors after running **CppLint**?)
cla-bot[bot] commented 3 years ago

If the initial contribution is required CLA agreement.

한국인은 국내법에 의해 개인정보제공동의가 필요합니다. pdf파일에 서명 후 oss@kakaocorp.com 으로 메일 보내 주세요.

krikit commented 3 years ago

@tooget

우선 수정해서 PR 주신데 대해 감사드립니다.

카카오에서는 국내 컨트리뷰터의 경우 개인정보동의가 필요합니다. 이는 국내법에 의한 것이며, 최초의 커밋에 한해 동의해 주시면 됩니다. 이후 커밋 부터는 필요 없는 절차입니다.

cla-bot의 글을 잘 보시고, 서명 후 메일로 제출을 부탁드립니다.

감사합니다.

tooget commented 3 years ago

@tooget

우선 수정해서 PR 주신데 대해 감사드립니다.

카카오에서는 국내 컨트리뷰터의 경우 개인정보동의가 필요합니다. 이는 국내법에 의한 것이며, 최초의 커밋에 한해 동의해 주시면 됩니다. 이후 커밋 부터는 필요 없는 절차입니다.

cla-bot의 글을 잘 보시고, 서명 후 메일로 제출을 부탁드립니다.

감사합니다.

@krikit 메일 송부 드렸습니다. 안내 감사합니다.

title: CLA(Kakao)_SUNGHYUN KIM_12-21-2020 12.25 to: oss@kakaocorp.com from: tooget@gmail.com

  • Khaiii가 향후 Apache License 2.0(혹은 MIT 등 불특정 다수가 특별한 제약 없이 사용할 수 있는 라이선스)를 유지하는 것을 전제로 CLA에 동의합니다.
oss-kakao commented 3 years ago

@cla-bot check

oss-kakao commented 3 years ago

@cla-bot check

cla-bot[bot] commented 3 years ago

If the initial contribution is required CLA agreement.

한국인은 국내법에 의해 개인정보제공동의가 필요합니다. pdf파일에 서명 후 oss@kakaocorp.com 으로 메일 보내 주세요.

cla-bot[bot] commented 3 years ago

The cla-bot has been summoned, and re-checked this pull request!

oss-kakao commented 3 years ago

@cla-bot check

cla-bot[bot] commented 3 years ago

The cla-bot has been summoned, and re-checked this pull request!

cla-bot[bot] commented 3 years ago

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Kim Sunghyun(SB テクノロジーユニット). This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email email@example.com
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails
tooget commented 3 years ago

.clabot 파일은 CLA(contributor license agreement)에 동의한 contributor를 저장하는 파일입니다. master에만 유지하기로 했는데요. 이 파일을 제거해 주시면 좋겠습니다.

@krikit 커멘트 감사드립니다. 파일 삭제하였습니다.

krikit commented 3 years ago

conflict가 나서 확인해 봤더니 도커 파일이 master와 develop의 상태가 달라서였네요. 도커 관련해서 여러 분들께서 도움을 주셨고, 지금은 처음의 도커 파일과 많이 다릅니다.

아마 @tooget 님은 master 브랜치를 사용하셨을 거고, 최신 변경은 develop 브랜치에 있습니다. 그래서 이 PR은 머지하기 힘들 것 같습니다. 처음부터 이 부분을 인지하지 못해서 괜히 시간을 허비하게 되어 죄송합니다.

저희는 develop -> master로 머지를 해서 안정적인 상태에서 태깅 후 배포하는 정책을 사용하고 있습니다. 그래서 master에는 develop의 내용이 배포되지 않으면 반영되지 않습니다. develop의 변경이 쌓여서 결국 master와의 간극이 많이 벌어진 상황입니다. 이 부분도 조만간 배포를 통해 해결할 수 있도록 노력하겠습니다.

tooget commented 3 years ago

@krikit

감사합니다.