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 e2e module to postgres example module #717

Merged
merged 4 commits into from
Dec 26, 2022

Conversation

mdelapenya
Copy link
Collaborator

@mdelapenya mdelapenya commented Dec 25, 2022

  • chore: move e2e tests to postgres example module
  • chore: bump postgres version to 14.1
  • chore: remove e2e module

What does this PR do?

It moves the tests that lived in the e2e module to the postgres example module.

Why is it important?

The e2e one was created with the sole goal to remove the dependency with postgres, and now that there is a specific module for postgres, it makes sense to merge both. From README in the e2e module:

Main purpose of this module is to have a separated module to avoid pushing
unexpected external dependencies like github.com/lib/pq into the main module.

@mdelapenya mdelapenya requested a review from a team as a code owner December 25, 2022 23:30
@mdelapenya mdelapenya added the dependencies Dependencies or external services label Dec 25, 2022
@mdelapenya mdelapenya self-assigned this Dec 25, 2022
@netlify
Copy link

netlify bot commented Dec 25, 2022

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit 4a7e768
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-go/deploys/63a94c5edc54d20008b10524
😎 Deploy Preview https://deploy-preview-717--testcontainers-go.netlify.app/examples/postgres
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@mdelapenya mdelapenya merged commit cc98efb into testcontainers:main Dec 26, 2022
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)
  ...
@mdelapenya mdelapenya deleted the remove-e2e-module branch June 23, 2023 05:30
This pull request was closed.
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.

1 participant