You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@daffodil.apache.org by "tuxji (via GitHub)" <gi...@apache.org> on 2023/02/03 16:13:27 UTC

[GitHub] [daffodil] tuxji commented on a diff in pull request #947: Add scalafmt (WIP)

tuxji commented on code in PR #947:
URL: https://github.com/apache/daffodil/pull/947#discussion_r1095984583


##########
daffodil-core/src/main/scala/org/apache/daffodil/dpath/DFDLExpressionParser.scala:
##########
@@ -17,8 +17,8 @@
 
 package org.apache.daffodil.dpath
 
-import java.math.{ BigDecimal => JBigDecimal }
-import java.math.{ BigInteger => JBigInt }
+import java.math.{BigDecimal => JBigDecimal}
+import java.math.{BigInteger => JBigInt}

Review Comment:
   Our Daffodil wiki's [Intellij IDEA Setup](https://cwiki.apache.org/confluence/display/DAFFODIL/IntelliJ+IDEA+Setup) has the following comment by Mike:
   
   > Some of IDEA's Scala settings are not consistent with our formatting standard. On the Spaces tab, check boxes for "Import braces".
   
   Therefore, is there a way to stop scalafmt from removing these whitespace characters from import braces?



-- 
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: commits-unsubscribe@daffodil.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org