empiredan commented on code in PR #1813: URL: https://github.com/apache/incubator-pegasus/pull/1813#discussion_r1436005687
########## scala-client/README.md: ########## @@ -53,3 +52,27 @@ It's built on top of the java client [apache/incubator-pegasus/java-client](http c.close ``` + +## Development + +### Format the code + +Use scala format tool, see https://github.com/scalameta/scalafmt +``` +sbt scalafmtSbt scalafmt test:scalafmt +``` + +### Run tests + +Build Java dependency at first, then build and test Scala client. +``` +cd ${PROJECT_ROOT}/java-client/scripts +./recompile_thrift.sh + +cd ${PROJECT_ROOT}/java-client +mvn clean package -DskipTests -Dcheckstyle.skip=true +mvn clean install -DskipTests -Dcheckstyle.skip=true + +cd ${PROJECT_ROOT}/scala-client +sbt test Review Comment: Before running test, should steps that build Pegasus and start onebox be added ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@pegasus.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@pegasus.apache.org For additional commands, e-mail: dev-h...@pegasus.apache.org