[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