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

fix: Add memcached container #322

Merged
merged 2 commits into from
May 11, 2024

Conversation

romulojales
Copy link
Contributor

@romulojales romulojales commented Mar 21, 2023

Adding Memcached container.

You have implemented a new container and would like to contribute it? Great! Here are the necessary steps.

  • Create a new feature directory and populate it with the package structure described in the documentation. Copying one of the existing features is likely the best way to get started.
  • Implement the new feature (typically in __init__.py) and corresponding tests.
  • Add a line -e file:[feature name] to requirements.in and run make requirements. This command will find any new requirements and generate lock files to ensure reproducible builds (see the pip-tools documentation for details). Then run pip install -r requirements/[your python version].txt to install the new requirements.
  • Update the feature README.rst and add it to the table of contents (toctree directive) in the top-level README.rst.
  • Add a line [feature name] to the list of components in the GitHub Action workflow in .github/workflows/main.yml to run tests, build, and publish your package when pushed to the master branch.
  • Rebase your development branch on master (or merge master into your development branch).

@romulojales romulojales marked this pull request as ready for review March 21, 2023 15:11
@damjankuznar
Copy link

damjankuznar commented Jun 22, 2023

@romulojales @pffijt Can you get this merged and released?

@romulojales
Copy link
Contributor Author

@romulojales @pffijt Can you get this merged and released?

Hello @damjankuznar ,

I just rebased the PR with the latest main version.

Since I don't manage the project, I'm also waiting for the maintainers to review, and hopefully, approve my pr, so I could work with it without using my own fork.

@alexanderankin alexanderankin added community-feat feature but its a community module so we wont bump tc core for it and removed 👀 requires attention labels Mar 26, 2024
@alexanderankin alexanderankin changed the title Add memcached container fix: Add memcached container May 11, 2024
@alexanderankin alexanderankin force-pushed the add-memcached branch 2 times, most recently from 9f9e66a to 9dadeb7 Compare May 11, 2024 04:24
@alexanderankin
Copy link
Collaborator

@alexanderankin alexanderankin merged commit 690b9b4 into testcontainers:main May 11, 2024
12 checks passed
alexanderankin pushed a commit that referenced this pull request May 14, 2024
🤖 I have created a release *beep* *boop*
---


##
[4.4.1](testcontainers-v4.4.0...testcontainers-v4.4.1)
(2024-05-14)


### Bug Fixes

* Add memcached container
([#322](#322))
([690b9b4](690b9b4))
* Add selenium video support
[#6](#6)
([#364](#364))
([3c8006c](3c8006c))
* **core:** add empty _configure to DockerContainer
([#556](#556))
([08916c8](08916c8))
* **core:** remove version from compose tests
([#571](#571))
([38946d4](38946d4))
* **keycloak:** add realm imports
([#565](#565))
([f761b98](f761b98))
* **mysql:** Add seed support in MySQL
([#552](#552))
([396079a](396079a))
* url quote passwords
([#549](#549))
([6c5d227](6c5d227))

---
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
community-feat feature but its a community module so we wont bump tc core for it ✨ package: new
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants