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.todayin multiple places here, I believe that is an anti-pattern that we should avoid. Suggesting refactoring allDate.todaytotodaywhich 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