tuxji commented on code in PR #947:
URL: https://github.com/apache/daffodil/pull/947#discussion_r1095998569
##########
daffodil-cli/src/main/scala/org/apache/daffodil/Main.scala:
##########
@@ -147,7 +146,10 @@ class CLIConf(arguments: Array[String]) extends
scallop.ScallopConf(arguments) {
}
}
}
- case _ => Left("you should provide no more than one argument for
this option") // Error because we expect there to be at most one flag
+ case _ =>
+ Left(
+ "you should provide no more than one argument for this option",
+ ) // Error because we expect there to be at most one flag
Review Comment:
> I've enabled rewrite.trailingCommas.style = always.
I suspect that scalafmt applies this parameter only to function call
arguments, not method parameters, because this is one of the only few places
I've seen a trailing comma added. Oh well, if that's all scalafmt can do,
that's okay.
This particular Left() call, BTW, was formatted oddly because there was a
long comment after the right parenthesis. This is one of the (I hope only a
few) places where we're going to have to move such a comment to its own line to
get the Left() back on one line, like this:
```scala
case _ =>
// Error because we expect there to be at most one flag
Left("you should provide no more than one argument for this
option")
```
--
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]
