[BUG] - Task not being called

[okridgway]

Describe the bug

I've written a simple program but can't for the life of me figure out why it isn't executing the task function.

To Reproduce

First I create a new scheduler:

	s := gocron.NewScheduler(time.UTC)

Then I specify how often the task should occur (every 10 seconds):

	s.Every(10).Do(task)

Finally, I start the task with:

	s.StartAsync()

But the task function (previously defined to print "Task is being performed" never executes. A playground link can be found here.

Version

I am using version v1.10.0.

Expected behavior

I expected the program to call the task function.

Additional context

I've tried adding a select{} statement at the end of my code as suggested here, but all to no avail.

[JohnRoesler]

Hi @okridgway

Running the playground link you provided I see it's going to always evaluate true as the scheduler s is not nil and it logs fatal outputting the memory addresses of the scheduler struct. I also don't see that the scheduler is being started.

Try this:

package main

import (
	"fmt"
	"log"
	"time"

	"github.com/go-co-op/gocron"
)

var task = func() {
	fmt.Println("Task is being performed.")
}

func main() {
	fmt.Println("Task is starting.")

	s := gocron.NewScheduler(time.UTC)
	_, err := s.Every(10).Do(task)
	if err != nil {
		log.Fatal(s)
	}

	s.StartBlocking()
}

It timed out for me in the playground, but runs locally just fine.

[okridgway]

Ok, I've got it working now (thank you!) but don't quite understand why s.Every(10).Do(task) has to be set to a variable in order for it to work, as the primary example on the main README suggests that it doesn't need to be set to a variable. Have things just changed? Or am I missing something else?

[JohnRoesler]

Can you paste the code you're running where you're seeing that have to set it to a variable? The README is correct.

For example, this works for me:

package main

import (
	"fmt"
	"time"

	"github.com/go-co-op/gocron"
)

var task = func() {
	fmt.Println("Task is being performed.")
}

func main() {
	fmt.Println("Task is starting.")

	s := gocron.NewScheduler(time.UTC)
	s.Every(10).Do(task)

	s.StartBlocking()
}

[JohnRoesler]

@okridgway I transferred your issue here as I deem it a discussion vs. a bug with gocron - I'm happy to continue helping here!