Skip to content

Add GitHub Actions CI workflow for Go - #3

Merged
kyleconroy merged 4 commits into
mainfrom
claude/add-go-workflow-GkYXc
Apr 26, 2026
Merged

Add GitHub Actions CI workflow for Go#3
kyleconroy merged 4 commits into
mainfrom
claude/add-go-workflow-GkYXc

Conversation

@kyleconroy

Copy link
Copy Markdown
Contributor

Summary

This PR adds a GitHub Actions continuous integration workflow to automatically build and test the Go project on push and pull request events.

Key Changes

  • Added .github/workflows/go.yml workflow file that:
    • Triggers on pushes and pull requests to main and master branches
    • Sets up Go 1.25 environment
    • Runs parser generation via make parser
    • Builds the project with go build ./...
    • Runs tests with go test ./...

Details

The workflow runs on the latest Ubuntu environment and uses standard GitHub Actions for Go setup. This ensures code quality and compatibility are validated automatically for all contributions.

https://claude.ai/code/session_01UZcF2NBhUS8AdGgBasRtF1

claude added 4 commits April 26, 2026 20:34
Runs `make parser` to generate the yacc-derived parser sources, then
`go build ./...` and `go test ./...` on push and pull_request.
The .y grammars and generated .go files now live in parser/, so update
the parser, generate, fmt, and clean targets to point at the new
locations. The %arser.go pattern rule didn't compose cleanly with a
directory prefix, so it's split into explicit parser/parser.go and
parser/hintparser.go rules. genkeyword is built into parser/ so the
//go:generate ./genkeyword directive in parser/generate.go resolves.
Regenerate parser.go and hintparser.go from the .y grammars on every
run, then `git diff --exit-code` against the working tree. If the
committed files don't match a fresh `make parser`, the job fails with
a hint to rerun and commit.
@kyleconroy
kyleconroy merged commit 77b6dcc into main Apr 26, 2026
1 check 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.

2 participants