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

Running 'go get -u -t ./...' #483

Closed
wants to merge 5 commits into from
Closed

Conversation

paketo-bot
Copy link
Member

No description provided.

@paketo-bot paketo-bot requested a review from a team as a code owner December 26, 2022 01:00
@paketo-bot paketo-bot added the semver:patch A change requiring a patch version bump label Dec 26, 2022
joshuatcasey
joshuatcasey previously approved these changes Jan 3, 2023
@joshuatcasey
Copy link
Contributor

This appears blocked on https://golang.testcontainers.org/quickstart/#2-install-testcontainers-for-go and paketo-buildpacks/packit#438.

When paketo-buildpacks/packit#438 is merged and released, I suggest closing this PR and rerunning the Go Get Update workflow to make sure go get -u -t ./... is run on latest main. Then add the replace directive found in https://golang.testcontainers.org/quickstart/#2-install-testcontainers-for-go and run go mod tidy and add that commit to the new PR.

@joshuatcasey joshuatcasey added the status/blocked This issue has been triaged and resolving it is blocked on some other issue label Jan 17, 2023
@robdimsdale
Copy link
Member

I'm going to close this and let the automation re-run the command against the latest changes on the main branch.

auto-merge was automatically disabled January 19, 2023 14:36

Pull request was closed

@robdimsdale robdimsdale deleted the automation/tools/go-get-update branch January 19, 2023 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver:patch A change requiring a patch version bump status/blocked This issue has been triaged and resolving it is blocked on some other issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants