[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