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

Add an optional "deps" paramter to the go/build pipeline. #428

Merged
merged 1 commit into from
May 5, 2023

Conversation

dlorenc
Copy link
Member

@dlorenc dlorenc commented May 5, 2023

We can use this to hoist up some of the imperative "go get'ing" we do to fix CVEs into a structured format.

We can use this to hoist up some of the imperative "go get'ing" we do to fix CVEs into
a structured format.

Signed-off-by: Dan Lorenc <dlorenc@chainguard.dev>
@dlorenc dlorenc requested a review from a team as a code owner May 5, 2023 12:12
@dlorenc dlorenc requested review from luhring and removed request for a team May 5, 2023 12:12
@dlorenc
Copy link
Member Author

dlorenc commented May 5, 2023

I've tested this against the existing pipelines that use go build and they don't break!

@rawlingsj rawlingsj enabled auto-merge May 5, 2023 12:17
@rawlingsj rawlingsj merged commit 08fae69 into chainguard-dev:main May 5, 2023
@@ -97,6 +97,10 @@ you can define the following values:
ldflags:
description:
List of [pattern=]arg to pass to the go compiler with -ldflags

deps:
Copy link
Member

Choose a reason for hiding this comment

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

Maybe update_deps?

@dlorenc dlorenc deleted the deps branch May 5, 2023 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants