Skip to content

Use porch setup-go-kpt composite action in CI workflows#757

Merged
efiacor merged 1 commit into
kptdev:mainfrom
Nordix:use-porch-go-setup-composite-action
May 29, 2026
Merged

Use porch setup-go-kpt composite action in CI workflows#757
efiacor merged 1 commit into
kptdev:mainfrom
Nordix:use-porch-go-setup-composite-action

Conversation

@aravindtga
Copy link
Copy Markdown
Contributor

Replace direct actions/setup-go usage and manual kpt installation with the kptdev/porch/.github/actions/setup-go-kpt composite action across all workflows.

Changes:

  • Use setup-go-kpt with install-kpt: 'false' for Go-only jobs
  • Bump actions/checkout to v6

AI usage disclosure

  • AI assistance (Kiro CLI) was used to draft portions of this change, and this PR description.

Signed-off-by: Aravindhan Ayyanathan <aravindhan.a@est.tech>
@aravindtga aravindtga marked this pull request as ready for review May 27, 2026 07:16
Copilot AI review requested due to automatic review settings May 27, 2026 07:16
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates the GitHub Actions CI workflow to use newer checkout and a custom Go setup action, simplifying the Go job’s checkout/layout.

Changes:

  • Bumps actions/checkout usage in CI jobs.
  • Reworks the Go job to use kptdev/porch/.github/actions/setup-go-kpt and a simplified module path.
  • Removes GOPATH-based checkout and runs the Go validation script from the default workspace root.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/ci.yaml
Comment thread .github/workflows/ci.yaml
@aravindtga aravindtga self-assigned this May 28, 2026
@efiacor efiacor merged commit 07f841a into kptdev:main May 29, 2026
8 checks passed
@efiacor efiacor deleted the use-porch-go-setup-composite-action branch May 29, 2026 08:29
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.

4 participants