Closed rjlohan closed 5 years ago
Looks like this depends on #18 . Should I just merge these changes into #19 , and enable
mvn compile test
? Then we could drop this PR completely.
Sounds good. Need another reviewer in here....
Done as part of #19
Description of changes: The
MainTest.java
file invokes the application and that isn't going to work on a shared repo. Will need fixup as we can.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.