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

chore: move redis-specific tests to the example module #701

Merged
merged 4 commits into from Dec 21, 2022

Conversation

mdelapenya
Copy link
Collaborator

  • chore: enhance redis example module's tests
  • chore: run go mod tidy in core module
  • chore: mod tidy redis example

What does this PR do?

It moves the redis-specific tests to the Redis example module, removing the import of the redis client in the core module

Why is it important?

Removing dependencies in the core module is a good thing :)

Related issues

@mdelapenya mdelapenya requested a review from a team as a code owner December 17, 2022 15:14
@mdelapenya mdelapenya added the dependencies Dependencies or external services label Dec 17, 2022
@mdelapenya mdelapenya self-assigned this Dec 17, 2022
@mdelapenya mdelapenya merged commit 593da80 into testcontainers:main Dec 21, 2022
mdelapenya added a commit to mdelapenya/testcontainers-go that referenced this pull request Dec 23, 2022
* main:
  docs: refine onboarding process with quickstart guide (testcontainers#706)
  chore: move redis-specific tests to the example module (testcontainers#701)
@mdelapenya mdelapenya deleted the remove-redis-dependency branch December 25, 2022 22:53
mdelapenya added a commit to mdelapenya/testcontainers-go that referenced this pull request Jan 4, 2023
* main: (44 commits)
  feat: support passing registry credentials to the reaper (testcontainers#647)
  fix: close response body in http strategy (testcontainers#718)
  chore: move e2e module to postgres example module (testcontainers#717)
  chore: bump containerd transitive dep in examples (testcontainers#715)
  chore(deps): bump github.com/containerd/containerd from 1.6.12 to 1.6.14 (testcontainers#703)
  chore(deps): bump github.com/compose-spec/compose-go in /modules/compose (testcontainers#710)
  chore: bump testcontainers-go to 0.17.0 in examples (testcontainers#714)
  chore(deps): bump github.com/docker/compose/v2 in /modules/compose (testcontainers#711)
  chore: support running MySQL compose in ARM (testcontainers#712)
  chore: simplify compose replace directives (testcontainers#713)
  chore: add compose module to dependabot (testcontainers#709)
  chore: move compose code to a separate module (testcontainers#650)
  docs: refine onboarding process with quickstart guide (testcontainers#706)
  chore: move redis-specific tests to the example module (testcontainers#701)
  chore: bump transitive dependencies (#527)
  chore: reduce concurrent builds (testcontainers#702)
  chore: add mysql example (testcontainers#700)
  chore(deps): bump google.golang.org/api from 0.104.0 to 0.105.0 (testcontainers#699)
  chore(deps): bump google.golang.org/api in /examples/firestore (testcontainers#683)
  chore(deps): bump cloud.google.com/go/spanner in /examples/spanner (testcontainers#688)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Dependencies or external services
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant