Skip to content

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

Closed
paketo-bot wants to merge 4 commits intomainfrom
automation/tools/go-get-update
Closed

Running 'go get -u -t ./...'#200
paketo-bot wants to merge 4 commits intomainfrom
automation/tools/go-get-update

Conversation

@paketo-bot
Copy link
Copy Markdown
Member

No description provided.

@paketo-bot paketo-bot requested a review from a team as a code owner December 26, 2022 01:10
@paketo-bot paketo-bot added the semver:patch A change requiring a patch version bump label Dec 26, 2022
Comment thread go.mod
require (
github.com/Azure/go-ansiterm v0.0.0-20210617225240-d185dfc1b5a1 // indirect
github.com/CycloneDX/cyclonedx-go v0.7.0 // indirect
github.com/CycloneDX/cyclonedx-go v0.7.1-0.20221222100750-41a1ac565cce // indirect
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems undesirable, although probably harmless.

Comment thread go.mod Outdated
github.com/sirupsen/logrus v1.9.0 // indirect
github.com/spdx/tools-golang v0.3.0 // indirect
github.com/spf13/afero v1.9.2 // indirect
github.com/spdx/tools-golang v0.3.1-0.20221108182156-8a01147e6342 // indirect
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems undesirable, although probably harmless.

@joshuatcasey
Copy link
Copy Markdown
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
Copy Markdown
Member

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

@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