Fix alignment of client built time in version output - #769
Conversation
Signed-off-by: Leander Janssen <leander@slaco.net>
Codecov Report
@@ Coverage Diff @@
## master #769 +/- ##
=======================================
Coverage 53.46% 53.46%
=======================================
Files 218 218
Lines 14642 14642
=======================================
Hits 7829 7829
Misses 6327 6327
Partials 486 486 |
thaJeztah
left a comment
There was a problem hiding this comment.
Ah! I think the alignment was due to a bug in the tabwriter, and this would not be the correct fix
ping @tiborvass PTAL
|
In GitHub this looks a bit weird, but in the actual code a single tab character is used. |
|
The tabwriter is meant to convert the tabs to spaces during presentation, so that the columns are always aligned properly. There was a bug in the tabwriter though, that caused the alignment to not work properly, so it was temporarily disabled (IIRC). I'm not exactly sure what the bug was, but @tiborvass mentioned that, so I guess he knows more. If that is resolved, alignment should work again (and this change would not be needed) |
|
Ah I see, I was actually quite surprised to see hard tabs in the code. |
|
ping @tiborvass |
|
Ugh, sorry for delay on this, thanks @leanderjanssen, but we need to use tabwriter here and I remember having some issues with it. If you want to tackle it let me know otherwise I'll try. |
vdemeester
left a comment
There was a problem hiding this comment.
LGTM 🐯
Let's go for that now and we'll see how to make it better later 👼
|
opened #965, using tabwriter |
- What I did
Add one more tab to the
Builttime to align the output of the docker client version.- How to verify it
Before:
After:
- Description for the changelog
Fix alignment of client built time in version output
Signed-off-by: Leander Janssen leander@slaco.net