Skip to content

fix(registry): module typed #811

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

Merged
merged 1 commit into from
Jun 15, 2025
Merged

fix(registry): module typed #811

merged 1 commit into from
Jun 15, 2025

Conversation

Tranquility2
Copy link
Contributor

@Tranquility2 Tranquility2 commented May 5, 2025

Discovered and supports #810

@Tranquility2 Tranquility2 changed the title Registry module typed fix(registry): module typed May 5, 2025
@max-pfeiffer
Copy link
Contributor

max-pfeiffer commented May 30, 2025

Hello @alexanderankin , I reviewed the code: it looks good to me. Any chance for you to do it as well soon?
I am asking because I have more improvements for that module which I would like to contribute.

@Tranquility2 Thanks for the good contribution.

@alexanderankin alexanderankin force-pushed the reg_type_sal branch 2 times, most recently from 5d80156 to a6d1f8b Compare June 15, 2025 06:35
@Tranquility2 Tranquility2 merged commit 6b11268 into main Jun 15, 2025
13 checks passed
@Tranquility2 Tranquility2 deleted the reg_type_sal branch June 15, 2025 06:56
alexanderankin pushed a commit that referenced this pull request Jun 16, 2025
🤖 I have created a release *beep* *boop*
---


##
[4.11.0](testcontainers-v4.10.0...testcontainers-v4.11.0)
(2025-06-15)


### Features

* **core:** Protocol support for container port bind and expose
([#690](#690))
([a0d4317](a0d4317))
* DockerContainer initializer to accept its private members as kwargs
([#809](#809))
([e7feb53](e7feb53))


### Bug Fixes

* **compose:** use provided docker command instead of default
([#785](#785))
([0ae704a](0ae704a))
* **core:** Add kwargs to image build
([#708](#708))
([cc02f94](cc02f94))
* **core:** change with_command type to include list of strings
([#789](#789))
([f7c29cb](f7c29cb))
* **core:** Determine docker socket for rootless docker
([#779](#779))
([6817582](6817582))
* **core:** Typing in docker_client
([#702](#702))
([e8bf224](e8bf224))
* **core:** Typing in generic + network
([#700](#700))
([2061912](2061912))
* **core:** Typing in version
([#701](#701))
([9dc2a02](9dc2a02))
* **core:** wait in test core registry
([#812](#812))
([b574c0e](b574c0e))
* **modules:** fix cosmosdb failure
([#827](#827))
([dafcbed](dafcbed))
* **modules:** update chroma version
([#826](#826))
([b7d41dd](b7d41dd))
* **rabbitmq:** correct pika pypi reference
([#817](#817))
([e90d308](e90d308))
* **registry:** module typed
([#811](#811))
([6b11268](6b11268))
* use connection mode override function in config
([#775](#775))
([ab2a1ab](ab2a1ab)),
closes
[#774](#774)

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

3 participants