[pbs-devel] applied: [PATCH v3 proxmox] time: drop trailing space when not showing seconds at end
Thomas Lamprecht
t.lamprecht at proxmox.com
Thu Oct 17 15:16:46 CEST 2024
Am 11/10/2024 um 11:33 schrieb Christian Ebner:
> Seconds are not displayed when the value is smaller than 0.1s and
> they are not at the start of the display output, e.g. `1h 2m`.
> Drop the additional whitespace currently appended for this edge case.
>
> Signed-off-by: Christian Ebner <c.ebner at proxmox.com>
> ---
> changes since version 2:
> - not present in previous version
>
> proxmox-time/src/time_span.rs | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
>
applied this one, thanks!
FYI: I found a bug within the Display code, months got printed with the
"m" unit, which is the unit for minutes. Besides fixing that I also made
it print "m" for minutes now to have more consistent unit variants (all
single letter) see the commit message for more details.
I also added a basic module documentation to convey what the format is and
where it comes from and some basic unit tests, could be surely expanded
though.
More information about the pbs-devel
mailing list