Changing required scala version to 2.11.5+ about twitter-korean-text HOT 12 CLOSED

확인해보니 scala-library를 2.11.5로 변경해도 에러메시지가 사라지지 않고, 2.12.0으로 올리니 없어졌습니다.

from twitter-korean-text.

스칼라 안정된 최신 버전은 2.11.7인데요, 그 버전을 써도 될까요?

from twitter-korean-text.

지금 2.11.7로 확인해보니 같은 에러메시지가 발생합니다.

from twitter-korean-text.

TwitterKoreanProcessorJava.tokensToJavaKoreanTokenList(tokens)이 자바 Token으로 바꾸어주기 때문제 가장 좋은 방법인 것 같습니다.

from twitter-korean-text.

scala-test가 아직 2.11 버전용 밖에 없어서 2.12로는 못 올릴 것 같습니다. ㅠㅜ

from twitter-korean-text.

그러면 KoreanTokenJava에 getter를 추가해주실 수 있으신가요?

말씀하신

TwitterKoreanProcessorJava.tokensToJavaKoreanTokenList(tokens)

를 사용하면, List<KoreanTokenJava>가 반환되는데요, KoreanTokenJava에는 getter가 없어서, 토큰의 KoreanPosJava pos를 읽어올 수 없습니다.

from twitter-korean-text.

https://github.com/twitter/twitter-korean-text/blob/master/src/main/java/com/twitter/penguin/korean/KoreanTokenJava.java#L39

KoreanTokenJava.getPos()를 만들어 놓은 것 같은데요, 다른 것이 필요한가요?

from twitter-korean-text.

이미 구현되어 있군요!
제가 README에 있는 4.0 버젼을 바라보고 있어서 그런 것 같습니다.
4.1.3으로 올려서 진행해보겠습니다.

from twitter-korean-text.

https://github.com/twitter/twitter-korean-text/pull/75/files 코드 리뷰 부탁드립니다.

from twitter-korean-text.

4.1.3으로 올려서 getter 확인 했습니다.
빠르게 답 주셔서 감사합니다!

from twitter-korean-text.

getter가 있어서, KoreanTokenJava의 별도 구현은 필요하지 않을 것 같습니다.
이 이슈도, 사실 KoreanTokenJava의 별도 구현 과정에서 발생한 이슈이기 때문에,
만약 (유닛테스트를 실패한다거나 하는) 문제가 발생한다면, scala-library의 버젼을 올리지 않으셔도 될 것 같습니다!

from twitter-korean-text.

감사합니다. 마침 트위터에서도 스칼라 버전을 업그레이드하기로 해서 2.11 버전이 필요하던 터였습니다. :)

from twitter-korean-text.