-
Notifications
You must be signed in to change notification settings - Fork 70
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New release with version using pgx v5? #66
Comments
This is a big issue for project using stdlib, I am getting an error "panic: sql: Register called twice for driver pgx" because obviously I cannot use stdlib/v4 and stdlib/v5 in a single binary. Unfortunately, we are still on the sqlx extension and we have to use: wrapErr := stdConn.Raw(func(pgxConn interface{}) error {
conn := pgxConn.(*stdlib.Conn).Conn()
opts := migrate.MigratorOptions{
MigratorFS: NewEmbeddedFS(&embeddedMigrations),
}
table := fmt.Sprintf("%s.schema_version", schema)
migrator, err := migrate.NewMigratorEx(ctx, conn, table, &opts) Many thanks! Appreciate the great work you do for the pgx community. |
You can use the v2-dev branch for pgx v5 compatibility. I don't anticipate any significant changes before the v2 release, but I want review the tern interface once more before a release. It may be a little while before I get to it though. |
For googlers: Unfortunately, it is not possible to work with branches in Go modules directly,
Ping me on github if you need me to push into the |
I just pinned this issue to the top so it gets more attention and tagged As mentioned above, there aren't any planned changes before final release, but more testing and feedback would be great. |
Thanks, there is one more little detail which prevents us from using the tag and that is: #69 Other than that, I believe it should work. |
Just tagged |
Thanks. Googlers: to try the new v2 beta version, do this:
And you are all set! |
I really like the API changes around LoadMigrations using a FS, before I had to create my own "migration filesystem". |
Finally released |
@meshenka Just release v2.2.0. |
Hi @jackc,
Any plans on releasing a new version soon? Thanks!
btw, I've tested it on my project (using the API rather than the CLI), and it worked fine!
The text was updated successfully, but these errors were encountered: