mike-mcgann commented on code in PR #919:
URL: https://github.com/apache/daffodil/pull/919#discussion_r1084353307
##########
xsrc/test/scripts/refactor/fix-imports.scala:
##########
@@ -0,0 +1,184 @@
+import java.io.BufferedWriter
+import java.io.FileWriter
+import java.io.File
+import scala.collection.mutable.ListBuffer
+import scala.collection.mutable.Map
+import scala.io.Source
+
+object FixImports extends App {
+ val inline = true
+ val importMap = Map(
+ "daffodil-cli" -> "cli",
+ "daffodil-core" -> "core",
+ "daffodil-io" -> "io",
+ "daffodil-lib" -> "lib",
+ "daffodil-macro-lib" -> "lib",
Review Comment:
I actually did that at first but the code that uses them expect the macros
to be in the same package and they are actually split across multiple packages
with `lib` just being one of them. This might require some changes to the
build.sbt. I think for this refactoring we should keep the functionality the
same, and if we want to change this, it could be done in a separate PR.
--
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]
