[GitHub] wujimin commented on issue #652: [SCB 456]Remove warning

2018-04-19 Thread GitBox
wujimin commented on issue #652: [SCB 456]Remove warning URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/652#issuecomment-382768277 fixed by https://github.com/apache/incubator-servicecomb-java-chassis/pull/660 but reviews have no any response, sigh..

[GitHub] wujimin commented on issue #652: [SCB 456]Remove warning

2018-04-17 Thread GitBox
wujimin commented on issue #652: [SCB 456]Remove warning URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/652#issuecomment-382082568 only one line changed, why so many commit log? you'd better use rebase not pull git command.

[GitHub] wujimin commented on issue #652: [SCB 456]Remove warning

2018-04-17 Thread GitBox
wujimin commented on issue #652: [SCB 456]Remove warning URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/652#issuecomment-382082058 it seems that you did not import intellij-java-google-style.xml This

[GitHub] wujimin commented on issue #652: [SCB 456]Remove warning

2018-04-17 Thread GitBox
wujimin commented on issue #652: [SCB 456]Remove warning URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/652#issuecomment-382075099 paste compile warning check image please This is an automated message