[bisq-network/bisq] Refactor CLI output formatting code & add trade formatter (#4703)
sqrrm
notifications at github.com
Fri Oct 30 13:00:01 UTC 2020
@sqrrm approved this pull request.
utACK
> +
+import com.google.common.annotations.VisibleForTesting;
+
+import java.util.function.Supplier;
+
+import static bisq.cli.ColumnHeaderConstants.*;
+import static bisq.cli.CurrencyFormat.formatOfferPrice;
+import static bisq.cli.CurrencyFormat.formatSatoshis;
+import static com.google.common.base.Strings.padEnd;
+
+ at VisibleForTesting
+public class TradeFormat {
+
+ @VisibleForTesting
+ public static String format(TradeInfo tradeInfo) {
+ // Some column values might be longer than header, so we need to calculated them.
```suggestion
// Some column values might be longer than header, so we need to calculate them.
```
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/4703#pullrequestreview-520639649
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20201030/c04c81d6/attachment.html>
More information about the bisq-github
mailing list