appease the linter and clear todos

Signed-off-by: gotjosh <[email protected]>
prometheus · Oct 12, 2023 · b1bfa40 · b1bfa40
1 parent db571c2
commit b1bfa40
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/notify/notify.go b/notify/notify.go
@@ -869,7 +869,6 @@ func (n SetNotifiesStage) Exec(ctx context.Context, l log.Logger, alerts ...*typ
 
 type timeStage struct {
 	muter types.TimeMuter
-	//Times map[string][]timeinterval.TimeInterval
 }
 
 type TimeMuteStage timeStage

diff --git a/timeinterval/timeinterval.go b/timeinterval/timeinterval.go
@@ -27,6 +27,8 @@ import (
 	"gopkg.in/yaml.v2"
 )
 
+// Intervener determines whether a given time and active route time interval should mute outgoing notifications.
+// It implements the TimeMuter interface.
 type Intervener struct {
 	intervals map[string][]TimeInterval
 }

diff --git a/types/types.go b/types/types.go
@@ -381,9 +381,8 @@ type Muter interface {
 	Mutes(model.LabelSet) bool
 }
 
-// TODO
+// TimeMuter determines if alerts should be muted based on the specified current time and active time interval on the route.
 type TimeMuter interface {
-	//TODO: I don't think this should return an error but let's keep it consistent for now.
 	Mutes(timeIntervalName []string, now time.Time) (bool, error)
 }