Skip to content

Commit

Permalink
Include unit when logging base delay option
Browse files Browse the repository at this point in the history
Just "Base delay: 40" is not very helpful. 40 what? Seconds?
Milliseconds? Martian days? Clarify that by including the unit.
  • Loading branch information
ruuda authored and OpsBotPrime committed Oct 14, 2021
1 parent 3efab01 commit 5c0407a
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/Config.hs
Original file line number Diff line number Diff line change
Expand Up @@ -148,7 +148,9 @@ instance Show (Options valid complete) where
, "Validate certs: " ++ showSpecified (oValidateCerts opts)
, "Inherit env: " ++ showSpecified (oInheritEnv opts)
, "Inherit env blacklist: " ++ showSpecified (oInheritEnvBlacklist opts)
, "Base delay: " ++ showSpecified (unMilliSeconds <$> oRetryBaseDelay opts)
, "Base delay: " ++ case oRetryBaseDelay opts of
Just (MilliSeconds ms) -> (show ms) ++ " ms"
Nothing -> "Unspecified"
, "Retry attempts: " ++ showSpecified (oRetryAttempts opts)
, "Log-level: " ++ showSpecified (oLogLevel opts)
, "Use PATH: " ++ showSpecified (oUsePath opts)
Expand Down

0 comments on commit 5c0407a

Please sign in to comment.