Skip to content

feat: Improve Go integrations#8

Open
jonathanio wants to merge 2 commits into
mainfrom
improve-go-integrations
Open

feat: Improve Go integrations#8
jonathanio wants to merge 2 commits into
mainfrom
improve-go-integrations

Conversation

@jonathanio
Copy link
Copy Markdown
Contributor

Improve the Go integrations and the GoReleaser processes to try and improve testing and building.

Checklist

Please check and confirm the following items have been performed, where
possible, for this Pull Request:

  • I have performed a self-review of my code and run any tests locally to check.
  • I have added tests that prove my changes are effective and work correctly.
  • I have made corresponding changes to the documentation as needed.
  • Each commit in, and this pull request, have meaningful subject & body for context.
  • I have added type/..., changes/..., and 'release/...' labels, as needed.

Improve the integrations through both Taskfiles and GitHub Workflows,
such as using tparse via the go:test task to improve layout and
summarisation of the tests in local developing, as well as build test
reports for tests via GitHub Workflows.

Also extend the coverage processing to create and upload test reports to
Codecov.
@jonathanio jonathanio self-assigned this May 17, 2026
Copilot AI review requested due to automatic review settings May 17, 2026 18:57
@jonathanio jonathanio added priority/normal This is a normal-priority issue or pull request release/update An update to an existing feature is made with this pull request update/configuration Update with changes to tool or service configuratons update/github-workflows Update with changes to GitHub Workflows update/go Update with changes to Go files or applications labels May 17, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 17, 2026

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@jonathanio jonathanio force-pushed the improve-go-integrations branch from e395a1e to 6f01394 Compare May 17, 2026 19:00
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

This PR updates the repository’s Go integration workflows and GoReleaser configuration to improve build/release automation (including container image signing/SBOM-related steps) and streamlines local Task task naming.

Changes:

  • Refactors Task task names (dropping the utils: prefix) and removes the GoReleaser “healthcheck” task dependency.
  • Expands .goreleaser.yaml with snapshot/source/SBOM/signing/docker release configuration and adds a new scratch-based Dockerfile.
  • Updates GitHub Actions workflows for Go integrations and coverage reporting (including Cosign installation).

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
Taskfile.yaml Updates develop task dependencies to renamed tasks.
.task/utils.yaml Renames utility tasks and keeps pre-check / pre-commit wiring.
.task/go.yaml Removes the go:healthcheck task and its dependency from builds.
internal/apply/apply.go Adjusts per-file processing logs and adds lint suppression for function length.
Dockerfile Adds a scratch image Dockerfile intended for GoReleaser multi-platform builds.
.goreleaser.yaml Major GoReleaser configuration changes: snapshot/source archives, checksums, SBOMs, signing, docker image publishing.
.github/workflows/go-integrations.yaml Updates GoReleaser invocation and installs Cosign in CI.
.github/workflows/coverage.yaml Adjusts Codecov upload ordering and adds additional (duplicated) test-results steps.
Comments suppressed due to low confidence (1)

.github/workflows/coverage.yaml:112

  • This second "Process test results" step will fail because tests.json was already removed by the earlier step (line 94). It also duplicates the upload logic (you already upload test results via codecov/test-results-action). Please remove the duplicated block or keep only one upload mechanism.

Comment thread internal/apply/apply.go Outdated
Comment thread internal/apply/apply.go
Comment thread Dockerfile
Comment thread .goreleaser.yaml
Comment thread .goreleaser.yaml Outdated
version: v2.15.4
args: >-
release
--clean
Comment thread .github/workflows/go-integrations.yaml Outdated
Improve the GoReleaser processes including support for building
container images, better selection of steps to run during local and
continuous integrations, and some initial attempts at signing and
attestations.
@jonathanio jonathanio force-pushed the improve-go-integrations branch from 6f01394 to 22b4a67 Compare May 17, 2026 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority/normal This is a normal-priority issue or pull request release/update An update to an existing feature is made with this pull request update/configuration Update with changes to tool or service configuratons update/github-workflows Update with changes to GitHub Workflows update/go Update with changes to Go files or applications

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants