Skip to content

feat: add GitHub Actions CI workflow#113

Merged
bborn merged 2 commits intomainfrom
task/218-add-ci-to-the-gh-repo
Jan 11, 2026
Merged

feat: add GitHub Actions CI workflow#113
bborn merged 2 commits intomainfrom
task/218-add-ci-to-the-gh-repo

Conversation

@bborn
Copy link
Copy Markdown
Owner

@bborn bborn commented Jan 11, 2026

Summary

  • Add GitHub Actions CI workflow that runs on push to main/master and pull requests
  • CI includes three parallel jobs: tests (with race detection), linting (golangci-lint), and build verification
  • Uses Go 1.24 to match the project's go.mod requirements

Test plan

  • Verify CI workflow triggers on this PR
  • Confirm all three jobs (test, lint, build) pass

🤖 Generated with Claude Code

Add CI configuration that runs on push to main/master and on pull requests:
- Test job: runs go test with race detection
- Lint job: runs golangci-lint
- Build job: verifies both binaries compile

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@openhands-ai
Copy link
Copy Markdown

openhands-ai bot commented Jan 11, 2026

Looks like there are a few issues preventing this PR from being merged!

  • GitHub Actions are failing:
    • CI

If you'd like me to help, just leave a comment, like

@OpenHands please fix the failing actions on PR #113 at branch `task/218-add-ci-to-the-gh-repo`

Feel free to include any additional details that might help me get this PR into a better state.

You can manage your notification settings

- Add .golangci.yml to configure linting rules:
  - Disable errcheck (codebase intentionally ignores some error returns)
  - Disable SA9003 (empty branch) and S1017 (TrimPrefix suggestion)
- Remove unused functions:
  - scpFile in cmd/task/cloud.go
  - createTask, retryTask in internal/ui/app.go
  - breakTmuxPane, killTmuxSession, humanizeTime in internal/ui/detail.go
  - killTmuxSession, isClaudeIdle, isWaitingForInput, hasCompletionMarker,
    parseOutputMarkers in internal/executor/executor.go
- Fix unused field: remove err from taskKilledMsg struct
- Fix ineffectual assignments in internal/ui/settings.go

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@bborn bborn merged commit 6f60d48 into main Jan 11, 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