Merge #420

420: Canonicalize some `Period`s and `CompoundPeriod`s when logging them with `@info` r=DilumAluthge a=DilumAluthge

Right now, if a PR has not yet met the minimum waiting period, we print the age like this:
```
pr_age = 1234567 milliseconds
```

This is not very helpful. So let's also use `Dates.canonicalize` to print the age in a more human-readable format.

Co-authored-by: Dilum Aluthge <[email protected]>

Author: bors[bot]

Project.toml

@@ -1,7 +1,7 @@
 name = "RegistryCI"
 uuid = "0c95cc5f-2f7e-43fe-82dd-79dbcba86b32"
 authors = ["Dilum Aluthge <[email protected]>", "Fredrik Ekre <[email protected]>", "contributors"]
-version = "7.1.7"
+version = "7.2.0-DEV"
 
 [deps]
 Base64 = "2a0f44e3-6c83-55bd-87e4-b1978d98bd5f"

src/AutoMerge/AutoMerge.jl

@@ -25,6 +25,7 @@ include("assert.jl")
 include("automerge_comment.jl")
 include("changed_files.jl")
 include("cron.jl")
+include("dates.jl")
 include("dependency_confusion.jl")
 include("github.jl")
 include("guidelines.jl")

src/AutoMerge/cron.jl

@@ -322,13 +322,13 @@ function cron_or_api_build(
                 "Reason: mandatory waiting period has not elapsed.",
             ),
             pr_type,
-            pr_age,
+            pr_age, _canonicalize_period(pr_age),
             pkg,
             is_jll_name(pkg),
-            new_package_waiting_period,
-            new_jll_package_waiting_period,
-            new_version_waiting_period,
-            new_jll_version_waiting_period,
+            new_package_waiting_period, _canonicalize_period(new_package_waiting_period),
+            new_jll_package_waiting_period, _canonicalize_period(new_jll_package_waiting_period),
+            new_version_waiting_period, _canonicalize_period(new_version_waiting_period),
+            new_jll_version_waiting_period, _canonicalize_period(new_jll_version_waiting_period),
             pr_author,
             authorized_authors,
             authorized_authors_special_jll_exceptions

src/AutoMerge/dates.jl

@@ -0,0 +1,7 @@
+function _canonicalize_period(p::Dates.CompoundPeriod)
+    return Dates.canonicalize(p)
+end
+
+function _canonicalize_period(p::Dates.Period)
+    return _canonicalize_period(Dates.CompoundPeriod(p))
+end

src/AutoMerge/public.jl

@@ -6,7 +6,7 @@ Run the `RegistryCI.AutoMerge` service.
 # Arguments
 
 - `env`: an `AbstractDictionary` used to read environmental variables from.
-   Defaults to `ENV` but a plain `Dict` can be passed to mimic an alternate environment. 
+   Defaults to `ENV` but a plain `Dict` can be passed to mimic an alternate environment.
 - `ciccfg`: Configuration struct describing the continuous integration (CI) environment in which AutoMerge is being run.
 
 # Keyword Arguments