This looks good.

But shouldn't we rather explore go:embed instead of reaching for .sql files to a file system?

But shouldn't we rather explore go:embed instead of reaching for .sql files to a file system?

The env variables are only relevant in main.go, where we compile the binaries and distribute them. In this form there is no go:embed.

But for folks that use goose as a library these env variables don't apply since they specify the embedded directory:

//go:embed migrations/*.sql
var embedMigrations embed.FS

...

goose.SetBaseFS(embedMigrations)
goose.Up(db, "migrations")

Does that sound about right?

Right! Yep all good, Mike :)