Rename it

errors.go

@@ -41,7 +41,7 @@ var (
 	ErrWithNameEmpty                 = fmt.Errorf("gocron: WithName: name must not be empty")
 	ErrWithStartDateTimePast         = fmt.Errorf("gocron: WithStartDateTime: start must not be in the past")
 	ErrWithStopTimeoutZeroOrNegative = fmt.Errorf("gocron: WithStopTimeout: timeout must be greater than 0")
-	ErrWithMonitorerNil              = fmt.Errorf("gocron: WithMonitorer: monitorer must not be nil")
+	ErrWithMonitorNil                = fmt.Errorf("gocron: WithMonitor: monitor must not be nil")
 )
 
 // internal errors

executor.go

@@ -23,7 +23,7 @@ type executor struct {
 	limitMode        *limitModeConfig
 	elector          Elector
 	locker           Locker
-	monitorer        Monitorer
+	monitor          Monitor
 }
 
 type jobIn struct {
@@ -353,18 +353,18 @@ func (e *executor) runJob(j internalJob, shouldSendOut bool) {
 
 	startTime := time.Now()
 	err := callJobFuncWithParams(j.function, j.parameters...)
-	if e.monitorer != nil {
-		e.monitorer.WriteTiming(startTime, time.Now(), j.id, j.name, j.tags)
+	if e.monitor != nil {
+		e.monitor.WriteTiming(startTime, time.Now(), j.id, j.name, j.tags)
 	}
 	if err != nil {
 		_ = callJobFuncWithParams(j.afterJobRunsWithError, j.id, j.name, err)
-		if e.monitorer != nil {
-			e.monitorer.Inc(j.id, j.name, j.tags, Fail)
+		if e.monitor != nil {
+			e.monitor.Inc(j.id, j.name, j.tags, Fail)
 		}
 	} else {
 		_ = callJobFuncWithParams(j.afterJobRuns, j.id, j.name)
-		if e.monitorer != nil {
-			e.monitorer.Inc(j.id, j.name, j.tags, Success)
+		if e.monitor != nil {
+			e.monitor.Inc(j.id, j.name, j.tags, Success)
 		}
 	}
 }

metrics.go

@@ -16,8 +16,8 @@ const (
 	Success JobStatus = "success"
 )
 
-// Monitorer represents the interface to collect jobs metrics.
-type Monitorer interface {
+// Monitor represents the interface to collect jobs metrics.
+type Monitor interface {
 	Inc(id uuid.UUID, name string, tags []string, status JobStatus)
 	WriteTiming(startTime, endTime time.Time, id uuid.UUID, name string, tags []string)
 }

scheduler.go

@@ -775,13 +775,13 @@ func WithStopTimeout(timeout time.Duration) SchedulerOption {
 	}
 }
 
-// WithMonitorer sets the metrics provider to be used by the Scheduler.
-func WithMonitorer(monitorer Monitorer) SchedulerOption {
+// WithMonitor sets the metrics provider to be used by the Scheduler.
+func WithMonitor(monitor Monitor) SchedulerOption {
 	return func(s *scheduler) error {
-		if monitorer == nil {
-			return ErrWithMonitorerNil
+		if monitor == nil {
+			return ErrWithMonitorNil
 		}
-		s.exec.monitorer = monitorer
+		s.exec.monitor = monitor
 		return nil
 	}
 }

scheduler_test.go

@@ -910,8 +910,8 @@ func TestScheduler_WithOptionsErrors(t *testing.T) {
 		},
 		{
 			"WithMonitorer nil",
-			WithMonitorer(nil),
-			ErrWithMonitorerNil,
+			WithMonitor(nil),
+			ErrWithMonitorNil,
 		},
 	}
 
@@ -1721,7 +1721,7 @@ func TestScheduler_WithMonitorer(t *testing.T) {
 		t.Run(tt.name, func(t *testing.T) {
 			ch := make(chan struct{}, 20)
 			testMonitorer := newTestMonitorer()
-			s, err := NewScheduler(WithMonitorer(testMonitorer))
+			s, err := NewScheduler(WithMonitor(testMonitorer))
 			require.NoError(t, err)
 
 			opt := []JobOption{