Use porch setup-go-kpt composite action in CI workflows#1253
Open
aravindtga wants to merge 2 commits into
Open
Use porch setup-go-kpt composite action in CI workflows#1253aravindtga wants to merge 2 commits into
aravindtga wants to merge 2 commits into
Conversation
✅ Deploy Preview for krm-function-catalog canceled.
|
Signed-off-by: Aravindhan Ayyanathan <aravindhan.a@est.tech>
120ee20 to
945cef2
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the repository’s GitHub Actions workflows to standardize Go (+ optional kpt) setup via the kptdev/porch/.github/actions/setup-go-kpt composite action, removing bespoke setup logic from individual jobs.
Changes:
- Replaces
actions/setup-goand manualgo install github.com/kptdev/kpt@...steps withsetup-go-kptacross CI/release workflows (disabling kpt install for Go-only jobs). - Updates CI checkout usage (including removing the legacy “checkout into GOPATH” pattern for unit tests).
- Bumps
actions/checkoutusage in the touched workflows to@v6.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| .github/workflows/ci.yaml | Switches CI jobs to use setup-go-kpt (with/without kpt) and updates checkout version. |
| .github/workflows/after-tag-with-version.yaml | Updates release-tag build workflow to use setup-go-kpt and newer checkout. |
| .github/workflows/after-push-to-branch.yaml | Updates main-branch build/push workflow to use setup-go-kpt and newer checkout. |
Comments suppressed due to low confidence (2)
.github/workflows/ci.yaml:60
- Using a Git ref of
@mainfor a third-party/composite GitHub Action is not immutable and can change unexpectedly (or be force-pushed), which can break CI and weakens supply-chain security. Prefer pinning to a versioned tag (e.g.@vX.Y.Z) or a full commit SHA.
- name: Set up Go
uses: kptdev/porch/.github/actions/setup-go-kpt@main
with:
go-version-file: documentation/go.mod
install-kpt: 'false'
.github/workflows/ci.yaml:76
- Using a Git ref of
@mainfor a third-party/composite GitHub Action is not immutable and can change unexpectedly (or be force-pushed), which can break CI and weakens supply-chain security. Prefer pinning to a versioned tag (e.g.@vX.Y.Z) or a full commit SHA.
- uses: actions/checkout@v6
- name: Set up Go and install kpt
uses: kptdev/porch/.github/actions/setup-go-kpt@main
with:
go-version-file: functions/go/list-setters/go.mod
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
liamfallon
reviewed
May 26, 2026
Signed-off-by: Aravindhan Ayyanathan <aravindhan.a@est.tech>
liamfallon
approved these changes
May 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Replace direct actions/setup-go usage and manual kpt installation with the
kptdev/porch/.github/actions/setup-go-kptcomposite action across all workflows.Changes:
install-kpt: 'false'for Go-only jobs (validate-metadata, go-unit-test-ci, build-push workflows)functions/go/list-setters/go.modin e2e-ci to resolve both Go and kpt versions from go.modAI usage disclosure