Skip to content

STAC-25142 Port CI to GitHub Actions (lint + test)#2

Merged
LouisParkin merged 1 commit into
masterfrom
STAC-25142-github-actions-ci
Jul 10, 2026
Merged

STAC-25142 Port CI to GitHub Actions (lint + test)#2
LouisParkin merged 1 commit into
masterfrom
STAC-25142-github-actions-ci

Conversation

@LouisParkin

Copy link
Copy Markdown
Contributor

What

Ports the .gitlab-ci.yml (lint + test) to GitHub Actions as part of the GitLab→GitHub migration. The GitLab mirror has been disabled, so GitHub is now canonical.

  • lint: gofmt check (fails on unformatted files) + revive (pinned v1.15.0) using the existing revive-recommended.toml
  • test: go test -v ./...

Why GitHub-hosted ubuntu-latest

This is a public Go library with no secrets, no image push, and no internal-registry needs. The GitLab job pulled golang:1.26 through the internal proxy on a self-hosted runner; on GitHub that's just actions/setup-go (Go version read from go.mod), and public repos get free GitHub-hosted minutes — no self-hosted runner required.

Notes

  • No import-path migration needed: go.mod is already github.com/StackVista/stackstate-receiver-go-client, so consumers already resolve via GitHub.
  • SHA-pinned actions, permissions: contents: read, persist-credentials: false. Zizmor clean.
  • revive run via go run …@v1.15.0 (pinned) instead of go install …@latest — reproducible and avoids PATH issues.

Jira

STAC-25142

Migrate the .gitlab-ci.yml lint+test pipeline to GitHub Actions as part of the
GitLab -> GitHub migration. Public Go library with no secrets/registry needs, so
it runs on GitHub-hosted ubuntu-latest with actions/setup-go (Go from go.mod):

- lint: gofmt check (fails on unformatted files) + revive (pinned v1.15.0) with
  the existing revive-recommended.toml
- test: go test -v ./...

SHA-pinned actions, minimal permissions, persist-credentials: false. Zizmor clean.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@LouisParkin
LouisParkin merged commit 64680ff into master Jul 10, 2026
2 checks passed
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.

1 participant