stevedlawrence commented on PR #1233: URL: https://github.com/apache/daffodil/pull/1233#issuecomment-2098184638
> Looks like 'sbt scalafmtAll scalafmtsbt' is not enough. There are still other files that this doesn't hit. Yep, for reference, the `daffodil-test-integration` project is not aggregated so that slow tests are not run when you call `sbt test`. But this skips all sbt commands for the subproject, not just `sbt test`, so all sbt commands (such as scalafmt) must be run specifically on that project with `sbt daffodil-test-integration/<command>`. Our CI does this when compile, run tests, and check scala formatting. Ideally we could just disable "test" aggregation for this subproject, but sbt doesn't provide that control. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
