Skip to content

Add basic Go linting and Go/frontend CI - #1

Merged
simonswine merged 1 commit into
mainfrom
ci/basic-checks
Oct 13, 2026
Merged

simonswine merged 1 commit into
mainfrom
ci/basic-checks

Conversation

@simonswine

Copy link
Copy Markdown
Owner

Summary

  • Check Go formatting and go vet; run the full Go suite with the race detector.
  • Install pinned frontend dependencies and run unit plus Chromium integration tests.
  • Rebuild frontend assets and fail if tracked assets differ or generated files are untracked.
  • Pin GitHub Actions to commit SHAs, use read-only permissions, and disable persisted checkout credentials.
  • Fix an existing gofmt issue and wait for asynchronous canvas sizing in browser tests.

Validation

  • nix run sys#zizmor -- .github/workflows/ci.yml: no findings
  • go vet ./...
  • go test -race ./... -count=1
  • Go formatting check
  • pnpm install --frozen-lockfile
  • pnpm test: 5 passed
  • pnpm test:integration: 3 passed
  • Rebuilt assets match committed files
  • git diff --check

@simonswine
simonswine merged commit 7e54639 into main Oct 13, 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