Remove unused else{} case
@@ -130,9 +130,9 @@ func (d *checkDelegate) WaitToRun(ctx context.Context, scope db.ResourceConfigSc
} else if !d.clock.Now().Before(runAt) {
// run if we're past the last check end time
shouldRun = true
- } else {
- // XXX(check-refactor): we could potentially sleep here until runAt is
- // reached.
+ }
+ // XXX(check-refactor): we could add an else{} case and potentially sleep
+ // here until runAt is reached.
//
// then the check build queueing logic is to just make sure there's a build
// running for every resource, without having to check if intervals have
@@ -141,7 +141,6 @@ func (d *checkDelegate) WaitToRun(ctx context.Context, scope db.ResourceConfigSc
// this could be expanded upon to short-circuit the waiting with events
// triggered by webhooks so that webhooks are super responsive: rather than
// queueing a build, it would just wake up a goroutine.
- }
if !shouldRun {
err := lock.Release()
