Skip to content

ci: add GitHub Actions CI workflow, issue templates, and PR template - #14

Merged
johnksterling merged 1 commit into
mainfrom
feature/ci-and-templates
Sep 14, 2026
Merged

johnksterling merged 1 commit into
mainfrom
feature/ci-and-templates

Conversation

@johnksterling

Copy link
Copy Markdown
Owner

Summary

  • GitHub Actions CI (.github/workflows/ci.yml):
    • Automatically runs on pushes and PRs to main.
    • Matrix testing across Go versions 1.21.x, 1.22.x, and 1.23.x.
    • Installs LilyPond in Ubuntu runner to test end-to-end PDF companion compilation.
    • Verifies code formatting (gofmt -l .), go vet, and tests with -race.
    • Performs CLI smoke tests for --help, analyze, and companion --pdf.
  • Cross-Platform LilyPond Test:
    • Updated lilypond_test.go to use CompileLilyPondToPDF instead of hardcoded macOS Homebrew path, enabling seamless execution on Linux CI runners.
  • Issue Templates (.github/ISSUE_TEMPLATE/):
    • Added structured YAML forms for bug_report.yml (tailored for harmonic analysis and MusicXML/iReal import quirks) and feature_request.yml.
    • Added config.yml linking to GitHub Discussions.
  • PR Template (.github/PULL_REQUEST_TEMPLATE.md):
    • Provides a clean checklist for verification and test coverage.

Verification

  • Local go test -v ./... passes 100%.
  • gofmt -l . clean.
  • go vet ./... clean.

- Add CI workflow testing Go 1.21, 1.22, 1.23 with LilyPond, gofmt check, and CLI smoke tests
- Update lilypond_test.go to use CompileLilyPondToPDF cross-platform
- Add structured issue templates for bug reports and feature requests
- Add PR template with verification checklist
@johnksterling
johnksterling merged commit f599a8b into main Sep 14, 2026
3 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