-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Task/psgs 44 sub 56 #13
Conversation
I need to add here it:test into CI, will do this today |
build.sbt
Outdated
dynverSonatypeSnapshots in ThisBuild := true, | ||
javacOptions ++= Seq("-source", "1.8", "-target", "1.8"), | ||
scalacOptions ++= Seq("-unchecked", "-deprecation", "-Ymacro-annotations"), | ||
parallelExecution in Test := false, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should be able to remove this for unit tests?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah ;) one sec
println(s"WALLET 1 words ${aryLen} <-") | ||
println(s"WALLET 2 words ${aryLen2} <-") | ||
println(s"WALLET 1 words $aryLen <-") | ||
println(s"WALLET 2 words $aryLen2 <-") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These can go. My bad.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(Well they were needed for debugging CI, but they are not needed anymore)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok will get rid of them
Integration tests in "it" folder, also moved by the way already apiExecutor so will be easier to create unit tests
( pointing PR to another branch, so only it test changes will be visible here )