-
-
Notifications
You must be signed in to change notification settings - Fork 17.7k
python3Packages.testcontainers: 4.10.0 -> 4.11.0 #417242
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
Conversation
b87648c to
bbf72c2
Compare
|
We cannot check the import of |
This breaks a ton of other packages. I wonder what we can do to prevent it from trying to open the socket. |
c31ccc4 to
ccbeba4
Compare
ccbeba4 to
99cced3
Compare
|
|
| # https://github.com/testcontainers/testcontainers-python/pull/832 | ||
| (fetchpatch { | ||
| name = "fix-get-docker-socket"; | ||
| url = "https://github.com/testcontainers/testcontainers-python/pull/832/commits/a6bcc12153c17b2aa498148af8b9a8258d86476b.patch"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This URL returns 406 for me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They ended up yanking the release. Let me mark the PR as a draft.
Things done
Diff: testcontainers/testcontainers-python@refs/tags/testcontainers-v4.10.0...testcontainers-v4.11.0
Changelog: https://github.com/testcontainers/testcontainers-python/releases/tag/testcontainers-v4.11.0
cc @onny
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Add a 👍 reaction to pull requests you find important.