-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
testcontainers/testcontainer-go has been renamed to testcontainers/testcontainers-go #11199
Comments
I ran into this too. I tried to create a PR to fix it but I get a weird |
I spent some time trying to get this to work and failed. The code using
|
@vangent I have seen similar issues before. I believe part of the problem is because there was some changes to how the checksum was calculated between certain Go 1.11 patch releases. My usual fix is to delete everything in |
Sorry, I'm not comfortable doing that for someone else's project. This should be a trivial change for a regular contributor; just needs to be updated to reference |
Fixes #11199 Signed-off-by: Leonardo Di Donato <leodidonato@gmail.com>
Fixes #11199 Signed-off-by: Leonardo Di Donato <leodidonato@gmail.com>
The correct solution here is to upgrade to the most recent version of Go (1.11.5), since old versions (1.11.0 - 1.11.3) calculated the checksum incorrectly. After upgrading, you probably need to run |
https://github.com/testcontainers/testcontainer-go has been renamed to https://github.com/testcontainers/testcontainers-go
I was having issues with Go modules and discovered that the
testcontainer-go
repo was renamed totestcontainers-go
while trying to importhashicorp/vault/api
whose repo requires the influxdb repo.I didn't have time to familiar myself with influxdb nor its code base, so I don't have a PR available. On the surface it doesn't seem to be a massive change, but I am not sure if
testcontainers-go
also broke other interfaces with this change.The text was updated successfully, but these errors were encountered: