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: run "go mod tidy" automatically when creating examples #794

Merged

Conversation

mdelapenya
Copy link
Collaborator

  • chore: do not generate go.sum in examples
  • chore: print success messages in main function
  • chore: run go mod tidy automatically

What does this PR do?

It removes the go.sum template as this PR is automating the execution of the "go mod tidy" command in the directory for the new example.

For that we leverage the exec package of stdlib, setting its directory to the new example dir. As a result, we are extracting the success message from the generate function, and instead moving it to the main one.

The success message has been changed in consequence

Why is it important?

Do even less extra work when generating an example!

@mdelapenya mdelapenya requested a review from a team as a code owner January 30, 2023 12:55
@mdelapenya mdelapenya added the chore Changes that do not impact the existing functionality label Jan 30, 2023
@mdelapenya mdelapenya self-assigned this Jan 30, 2023
@netlify
Copy link

netlify bot commented Jan 30, 2023

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit 03981a8
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-go/deploys/63d7ee0c78b01200083f8678
😎 Deploy Preview https://deploy-preview-794--testcontainers-go.netlify.app
📱 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.

examples/main.go Outdated Show resolved Hide resolved
examples/main.go Outdated Show resolved Hide resolved
@mdelapenya mdelapenya merged commit 898a14d into testcontainers:main Jan 30, 2023
@mdelapenya mdelapenya deleted the improve-experience-examples branch January 30, 2023 20:55
mdonkers pushed a commit to mdonkers/testcontainers-go that referenced this pull request Jan 31, 2023
…n-failure-or-exit

* origin/main:
  docs: add intel as user (testcontainers#798)
  chore: bump containerd in examples (testcontainers#797)
  chore(deps): bump github.com/containerd/containerd from 1.6.15 to 1.6.16 (testcontainers#793)
  chore: extract docker host calculation to an internal package (testcontainers#796)
  chore: run "go mod tidy" automatically when creating examples (testcontainers#794)
  chore: build images with backoff retries (testcontainers#792)
mdelapenya added a commit to mdelapenya/testcontainers-go that referenced this pull request Feb 10, 2023
* main:
  chore: update Docker labels for containers (testcontainers#813)
  fix: nil pointer dereference in HealthStrategy (testcontainers#802)
  fix: Synchronise writes to containers map (testcontainers#812)
  chore(deps): bump google.golang.org/api from 0.108.0 to 0.109.0 in /examples (testcontainers#810)
  chore(deps): bump cloud.google.com/go/spanner in /examples/spanner (testcontainers#806)
  chore: restructure Docker helper methods (testcontainers#799)
  Verify Reaper state to create new or return existing instance (testcontainers#782)
  docs: add intel as user (testcontainers#798)
  chore: bump containerd in examples (testcontainers#797)
  chore(deps): bump github.com/containerd/containerd from 1.6.15 to 1.6.16 (testcontainers#793)
  chore: extract docker host calculation to an internal package (testcontainers#796)
  chore: run "go mod tidy" automatically when creating examples (testcontainers#794)
  chore: build images with backoff retries (testcontainers#792)
  fix: use right import package for compose in docs (testcontainers#791)
  chore(deps): bump google.golang.org/grpc from 1.52.1 to 1.52.3 in /examples (testcontainers#790)
  Add devcontainer file (testcontainers#765)
  chore: check dependabot dependencies weekly (testcontainers#789)
  chore(deps): bump google.golang.org/grpc from 1.52.0 to 1.52.1 in /examples (testcontainers#783)
  chore: support for titles in examples (testcontainers#775)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Changes that do not impact the existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants