8000 feat: reusable containers by matthiasschaub · Pull Request #1 · GIScience/testcontainers-python · GitHub
[go: up one dir, main page]
Skip to content

feat: reusable containers #1

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.

Sign up for GitHub

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

Jump to bottom
Open
matthiasschaub wants to merge 25 commits into
base: main
Choose a base branch
from
Open

feat: reusable containers #1

matthiasschaub wants to merge 25 commits into from
+228 −4

Conversation

@matthiasschaub
Copy link
Member
@matthiasschaub matthiasschaub commented Jul 2, 2024

No description provided.

@matthiasschaub matthiasschaub changed the title feat: reuse containers feat: reusable containers Jul 3, 2024
@matthiasschaub matthiasschaub force-pushed the reusable_containers branch 2 times, most recently from 44660f1 to 74c8df1 Compare July 3, 2024 12:04
matthiasschaub and others added 18 commits July 8, 2024 09:40
@matthiasschaub
refactor: remove TODO comment
c656660
@matthiasschaub
docs: update section on reusable containers
efb1265
@matthiasschaub
feat(reuse): do not change contract of stop method
d4445d6
@matthiasschaub
feat(reuse): do not create Ryuk cleanup instance
1ea9ed1 
do not create Ryuk cleanup instance if reuse enabled and container has
been start with `with_reuse`
@matthiasschaub
refactor: move hash generation into if clause
ea6fec7
@matthiasschaub
Merge remote-tracking branch 'origin/main' into reusable_containers
7c1e8e7
@matthiasschaub
Merge branch 'main' into reusable_containers
2113561
@matthiasschaub
Merge branch 'main' into reusable_containers
0615c29
@matthiasschaub
Merge branch 'main' into reusable_containers
23e436a
@matthiasschaub
fix: revert accidental removal of @Property
2bfb36d
@matthiasschaub
Merge remote-tracking branch 'testcontainers/main' into reusable_cont…
6796a9a 
…ainers
@matthiasschaub
Merge remote-tracking branch 'testcontainers/main' into reusable_cont…
3050fee 
…ainers
@matthiasschaub
Merge branch 'main' into reusable_containers
beac036
@matthiasschaub
tests: access tc properties through function call
35e0599 
and assert not none to make mypy happy.
@matthiasschaub
refactor: use crc32 instead of sha1 to compute hash
e84fe02 
To compute a hash to identify container across runs use crc32 instead of
sha1 which is faster while still being fit-for-purpose.
@matthiasschaub
refactor: remove remnant code from past merge
3a31883
@matthiasschaub
docs: more detailed explanation on how to use reusable containers
4c0ba18
@matthiasschaub
fix: check if self._kwargs is None before using it
3894eb1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@matthiasschaub
0