Skip to content
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

Update sqlmock dependency #34

Merged
merged 1 commit into from
Apr 6, 2019
Merged

Update sqlmock dependency #34

merged 1 commit into from
Apr 6, 2019

Conversation

emersion
Copy link
Contributor

@emersion emersion commented Apr 6, 2019

go-sqlmock have switched from gopkg.in to Go modules. This repository uses the
new github.com import path, however the go.mod file still uses the old gopkg.in
import path. This confuses go mod when importing this package:

go: gopkg.in/DATA-DOG/go-sqlmock.v1@v1.3.3: go.mod has non-....v1 module path "github.com/DATA-DOG/go-sqlmock" at revision v1.3.3

This commit updates the go.mod import path.

(Please tag a new minor version after merging this)

go-sqlmock have switched from gopkg.in to Go modules. This repository uses the
new github.com import path, however the go.mod file still uses the old gopkg.in
import path. This confuses `go mod` when importing this package:

    go: gopkg.in/DATA-DOG/go-sqlmock.v1@v1.3.3: go.mod has non-....v1 module path "github.com/DATA-DOG/go-sqlmock" at revision v1.3.3

This commit updates the go.mod import path.
@emersion
Copy link
Contributor Author

emersion commented Apr 6, 2019

Note: the breakage has been caused by #33

@lucasb-eyer lucasb-eyer merged commit 30298f2 into lucasb-eyer:master Apr 6, 2019
@lucasb-eyer
Copy link
Owner

This is unfortunate, especially as sqlmock is only barely used for testing. I'm not up-to-date anymore and have to rely on PRs, so thanks a lot for fixing this!

@emersion emersion deleted the mod-update branch April 7, 2019 06:26
@emersion
Copy link
Contributor Author

emersion commented Apr 7, 2019

Thanks for merging this! Could you tag a new minor release so we can use your library without hitting errors?

@lucasb-eyer
Copy link
Owner

Thanks for reminding me, here it is: https://github.com/lucasb-eyer/go-colorful/releases/tag/v1.0.2
Let me know if it still doesn't work/I still forgot something :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants