search

jam2in / arcus-java-client

Arcus Java client
Apache License 2.0
0 stars 0 forks source link

move matchStatus line in else condition #16

Closed whchoi83 closed 8 years ago

whchoi83 commented 8 years ago

아래 이슈 수정사항 입니다. https://github.com/jam2in/arcus-java-client/issues/14

matchStatus 의 호출 위치가 잘못되어 불필요한 로그가 출력되는 것을 수정하기 위해 호출 위치를 변경하고, 중복된 if 조건의 내용을 하나로 합쳤습니다.

리뷰 부탁드립니다.

whchoi83 commented 8 years ago

ci test 에러는 time out 관련입니다. 로컬 환경에서 통과되는 것은 확인했습니다.

aiceru commented 8 years ago

review 완료했습니다.

@whchoi83 wooseok/fix_testTimeout 브랜치를 merge 한 다음 push 해서 CI test 결과를 보고 싶은데, 괜찮을까요? (이 이슈와는 별개의 건이긴 한데... 기회가 있을 때마다 최대한 CI test 를 많이 돌려보고 싶어서.. ^^;;)

jhpark816 commented 8 years ago

review 완료

whchoi83 commented 8 years ago

@aiceru 별 문제 없을 것 같습니다.

whchoi83 commented 8 years ago

merge 완료했습니다.