ci: migrate GHA to task, remove duplicate unit test job#223
Merged
Conversation
Add test:fuzz and cross-compilation support to build:shim in Taskfile.yml. Update fuzz and build-shim workflows to use task. Remove the duplicate tests job introduced alongside the fuzz job. Signed-off-by: Derek McGowan <derek@mcg.dev>
There was a problem hiding this comment.
Pull request overview
This PR consolidates GitHub Actions CI logic around task by moving fuzz testing and shim cross-compilation steps into Taskfile.yml, and removing a redundant unit test job from the workflow configuration.
Changes:
- Extend
build:shimto support cross-compilation viaGOOS/GOARCHoverrides. - Add a
test:fuzztask and switch the CI fuzz workflow to invoke it viatask. - Remove a duplicate unit test job from CI and update the shim build workflow to use
task build:shim.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| Taskfile.yml | Adds cross-compilation support for shim builds and introduces a reusable fuzz-testing task. |
| .github/workflows/ci.yml | Updates fuzz job to run via task and removes the redundant unit test job block. |
| .github/workflows/build-shim.yml | Migrates shim build steps to task build:shim with GOOS/GOARCH provided by the matrix. |
Comments suppressed due to low confidence (1)
.github/workflows/ci.yml:151
- Removing the dedicated unit test job also removes the only
go test -raceexecution from CI (there are no remaining -race invocations in this workflow). If the intent was only to remove duplication, consider keeping a race-detector run (e.g., on ubuntu-latest) to avoid losing coverage for data races.
#
# Fuzz tests
#
fuzz:
name: Fuzz Tests
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
austinvazquez
approved these changes
Jun 9, 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.
Add test:fuzz and cross-compilation support to build:shim in Taskfile.yml. Update fuzz and build-shim workflows to use task. Remove the duplicate tests job introduced alongside the fuzz job.