yogthos / migratus

MIGRATE ALL THE THINGS!
641 stars 95 forks source link

[BUG] up / down don't appear to be working with code-based migrations #239

Closed sirmspencer closed 1 year ago

sirmspencer commented 1 year ago

code-based migrations are super cool by the way. I have everything set up and running migrate picks up on the code-based migrations. up and don't don't seem to. I am using the same config, etc as our SQL migrations that do work with up and down.

yogthos commented 1 year ago

Have you added the up/down fn declarations?

{:ns app.migrations.import-users
 :up-fn migrate-up
 :down-fn migrate-down
 :transaction? true}
sirmspencer commented 1 year ago

Yes. And migratus/migrate works by calling the up function.

sirmspencer commented 1 year ago

Its working now. Not sure what happened.

yogthos commented 1 year ago

Hmm, maybe stale repl state.