Follow-up from "Enable health status automation for all deliverables"
The following discussion from !1879 (merged) should be addressed:
-
@jennli started a discussion: nitpick:
I just noticed we are calling
Date.today
in multiple places here, I believe that is an anti-pattern that we should avoid. Suggesting refactoring allDate.today
totoday
which is an available helpe!has_unique_comment?("triage-ops set-health-status-on-track-#{today.year}-#{today.month}")
But this is not changed in this MR so I'm going to resolve it for now and propose a follow-up issue for refactoring
Edited by Jennifer Li