-
-
Notifications
You must be signed in to change notification settings - Fork 363
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
build.mill files compiled by Scala 3 #3369
base: main
Are you sure you want to change the base?
Conversation
Thanks @bishabosha ! Lets get your mainargs changes landed, that way we can cut a release and you can test your WIP in CI |
+1 for switching over to using directives! |
9d1a986
to
dc71022
Compare
[Edit: after switching to depending on mainargs 0.7.2] in https://github.com/com-lihaoyi/mill/actions/runs/10418081276/job/28853542122?pr=3369 you can see integration tests like |
729c575
to
e69f1bd
Compare
Rebased and included the new mill-moduledefs 0.11.0-M1 |
a06daca
to
37ba541
Compare
917a024
to
78beea1
Compare
@lihaoyi just recording here that the mockito third party test can be flaky (specifically as seen in the CI run https://github.com/com-lihaoyi/mill/actions/runs/10909315276/job/30277476457?pr=3369#step:9:4941 |
022c789
to
368fec9
Compare
Seems like some of the new |
Seems like the Scala 3.6.2 upgrade is causing new issues, may be worth punting on that for now and just getting onto 3.5.x first |
This reverts commit 6e0709e.
Somehow it is this line that is causing the bootstrap compile to hang --- a/main/codesig/src/ReachabilityAnalysis.scala
+++ b/main/codesig/src/ReachabilityAnalysis.scala
@@ -91,7 +91,7 @@ class CallGraphAnalysis(
case None => ujson.Obj()
}
- logger.mandatoryLog(spanningInvalidationTree)
+ //logger.mandatoryLog(spanningInvalidationTree)
}
|
Looks like bootstrapping on Scala 3.6.2 works again after optimizing |
This is work in progress to fix #3152
Numerous fixes were also needed to com-lihaoyi/mainargs, com-lihaoyi/sourcecode, and com-lihaoyi/mill-moduledefs
With the current state, only 1 example/integration tests is still failing:
integration.feature[plugin-classpath].local
known TODOs:
Fix BSP reporter linenumbers for build scripts(Zinc reporter forwards to bsp)known incompatibilities:
ExplicitResultTypes
scalafix rule - need to upgrade mill-scalafixupgraded Scalafmt so not skipped anymoreCross.scala
uses the new quoted type syntax which scalafmt crashes on, (and version is frozen) so skip the fileexample.thirdparty[mockito]
integration.feature[plugin-classpath]
due to third party plugin dep