Skip to content
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

Make run command arguments Tasks #2452

Merged
merged 22 commits into from
May 1, 2023
Prev Previous commit
Next Next commit
update-patch
  • Loading branch information
lihaoyi committed Apr 29, 2023
commit 75532bc1ad6744d960ee2a5116698f0e276496b9
13 changes: 12 additions & 1 deletion ci/mill-bootstrap.patch
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
diff --git a/build.sc b/build.sc
index b6b06b7584..b49a252914 100644
index b6b06b7584..8c62146aef 100644
--- a/build.sc
+++ b/build.sc
@@ -2,34 +2,26 @@
Expand Down Expand Up @@ -438,6 +438,17 @@ index b6b06b7584..b49a252914 100644

def moduleDeps = Seq(scalalib, scalajslib, scalanativelib, runner.test)

@@ -1681,8 +1443,8 @@ object dev extends MillModule {
mill.modules.Jvm.createJar(Agg(), mill.modules.Jvm.JarManifest(manifestEntries))
}

- def run(args: String*) = T.command {
- args match {
+ def run(args: Task[Args] = T.task(Args())) = T.command {
+ args().value match {
case Nil => mill.api.Result.Failure("Need to pass in cwd as first argument to dev.run")
case wd0 +: rest =>
val wd = os.Path(wd0, T.workspace)
@@ -1709,67 +1471,11 @@ object docs extends Module {
def moduleDeps = build.millInternal.modules.collect { case m: MillApiModule => m }

Expand Down