fix all linting except unimplemented daily job

example_test.go

@@ -8,7 +8,7 @@ import (
 	"github.com/google/uuid"
 	"github.com/jonboulle/clockwork"
 
-	. "github.com/go-co-op/gocron/v2"
+	. "github.com/go-co-op/gocron/v2" // nolint:revive
 )
 
 func ExampleAfterJobRuns() {
@@ -160,7 +160,7 @@ func ExampleScheduler_NewJob() {
 		),
 	)
 	if err != nil {
-		// handle error
+		panic(err)
 	}
 	fmt.Println(j.ID())
 }

job.go

@@ -29,7 +29,6 @@ type internalJob struct {
 	timer            clockwork.Timer
 	singletonMode    bool
 	limitRunsTo      *limitRunsTo
-	lockerKey        string
 	startTime        time.Time
 	startImmediately bool
 	// event listeners
@@ -209,7 +208,7 @@ func (d durationRandomJobDefinition) setup(j *internalJob, _ *time.Location) err
 	j.jobSchedule = &durationRandomJob{
 		min:  d.min,
 		max:  d.max,
-		rand: rand.New(rand.NewSource(time.Now().UnixNano())),
+		rand: rand.New(rand.NewSource(time.Now().UnixNano())), // nolint:gosec
 	}
 	return nil
 }
@@ -247,7 +246,7 @@ func (d dailyJobDefinition) options() []JobOption {
 }
 
 func (d dailyJobDefinition) setup(i *internalJob, location *time.Location) error {
-	//TODO implement me
+	// TODO implement me
 	panic("implement me")
 }
 
@@ -280,10 +279,8 @@ func (w weeklyJobDefinition) setup(j *internalJob, location *time.Location) erro
 		return ErrWeeklyJobAtTimesNil
 	}
 
-	var daysOfTheWeek []time.Weekday
-	for _, wd := range w.daysOfTheWeek() {
-		daysOfTheWeek = append(daysOfTheWeek, wd)
-	}
+	daysOfTheWeek := w.daysOfTheWeek()
+
 	slices.Sort(daysOfTheWeek)
 
 	atTimesDate, err := convertAtTimesToDateTime(w.atTimes, location)
@@ -641,7 +638,7 @@ type dailyJob struct {
 }
 
 func (d dailyJob) next(lastRun time.Time) time.Time {
-	//TODO implement me
+	// TODO implement me
 	panic("implement me")
 }
 

job_test.go

@@ -8,8 +8,8 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
-func TestDurationJob_next(t *testing.T) {
-}
+//func TestDurationJob_next(t *testing.T) {
+//}
 
 func TestWeeklyJob_next(t *testing.T) {
 	tests := []struct {

scheduler.go

@@ -152,9 +152,8 @@ func (s *scheduler) stopScheduler() {
 	s.started = false
 	for id, j := range s.jobs {
 		j.stop()
-		select {
-		case <-j.ctx.Done():
-		}
+		<-j.ctx.Done()
+
 		j.ctx, j.cancel = context.WithCancel(s.shutdownCtx)
 		s.jobs[id] = j
 	}
@@ -318,9 +317,7 @@ func (s *scheduler) addOrUpdateJob(id uuid.UUID, definition JobDefinition) (Job,
 	} else {
 		currentJob := requestJob(s.shutdownCtx, id, s.jobOutRequestCh)
 		s.removeJobCh <- id
-		select {
-		case <-currentJob.ctx.Done():
-		}
+		<-currentJob.ctx.Done()
 
 		j.id = id
 	}