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

Need to make sure the JDK environment #79

Closed
gamamoe opened this issue Dec 16, 2019 · 3 comments
Closed

Need to make sure the JDK environment #79

gamamoe opened this issue Dec 16, 2019 · 3 comments
Assignees
Labels
need follow-up by maintainers

Comments

@gamamoe
Copy link
Member

gamamoe commented Dec 16, 2019

주제

현재 묵시적으로 KOMORAN은 JDK8 기준의 빌드 및 사용을 권장하는 것으로 알고 있습니다 (또는 JDK7)
빌드 환경에 대한 명시적인 설명 및 코드 베이스 정리가 필요해보입니다

선택지

  1. 메뉴얼에 빌드 환경 명시
  2. JDK8로 진행할 경우 빌드가 깨지는 테스트 케이스 수정

추가 정보

Test Class 중 JDK11에 도입된 strip() 메소드가 현재 JDK8 환경에서는 제대로 빌드되지 않음

https://github.com/shin285/KOMORAN/blob/master/src/test/java/kr/co/shineware/nlp/komoran/core/SejongToTrainingData.java#L85

@gamamoe gamamoe added the need follow-up by maintainers label Dec 16, 2019
@9bow
Copy link
Member

9bow commented Dec 16, 2019

Java 8에 맞추는 것으로 하고, strip()trim() 또는 커스텀 메소드로 변경하는건 어떨까요?

@shin285
Copy link
Member

shin285 commented Dec 16, 2019

trim()을 썼어야 했는데 strip()을 썼네요. 메소드는 일단 trim()으로 변경해놓겠습니다. 저 test class는 ignore가 되어 있어서 trim으로 변경되면 문제가 되지 않을 것으로 예상됩니다. 테스트 해보고 해당 이슈 처리하겠습니다.

@gamamoe
Copy link
Member Author

gamamoe commented Dec 30, 2019

#80 resolved this issue

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

No branches or pull requests

3 participants