Skip to content

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 all Date.today to today 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