-
-
Notifications
You must be signed in to change notification settings - Fork 401
Add examples of mill + cask + scalasql #4924
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
base: main
Are you sure you want to change the base?
Add examples of mill + cask + scalasql #4924
Conversation
@@ -0,0 +1,265 @@ | |||
#!/usr/bin/env sh |
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.
Shouldn't need the mill
file in the examples, it gets filled in automatically when publishing
ivy"com.h2database:h2:2.2.224" | ||
) | ||
|
||
object test extends ScalaTests { |
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.
Let's have one test
suite using H2 in memory and one integration
suite using PostgreSQL via testcontainers.
@chaitanyawaikar left some comments but otherwise looks reasonable |
"[6549-0] [error] -- [E8] /home/runner/work/mill/mill/out/example/scalalib/web/11-todo-http4s-scalasql/native/server/testForked.dest/worker-0/sandbox/run-2/build.mill:7:5 [6549-0] [error] 7 ? ivy"org.http4s::http4s-ember-server::0.23.30", [6549-0] [error] ? ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [6549-0] [error] ?value ivy is not a member of StringContext, but could be made available as an extension method. [6549-0] [error] ? [6549-0] [error] ?One of the following imports might fix the problem: [6549-0] [error] ? [6549-0] [error] ? import coursier.ivyRepositoryString [6549-0] [error] ? import coursier.util.StringInterpolators.SafeIvyRepository [6549-0] [error] ? [6549-0] [error] ? [6549-0] [error] -- [E8] /home/runner/work/mill/mill/out/example/scalalib/web/11-todo-http4s-scalasql/native/server/testForked.dest/worker-0/sandbox/run-2/build.mill:8:5"
Description
This PR solves the issue #4799. It includes creation of two new examples of the very popular TODO MVC app
10-todo-webapp-cask-scalasql
11-todo-http4s-scalasql
Unit tests and a short documentation have been added for each of these modules.
@lihaoyi Once the approach/implementation looks fine, I can proceed to adding the integration tests.