Skip to content

Run duneapi-client-go tests in CI#29

Merged
charisra merged 2 commits into
mainfrom
charis/gro-429-run-duneapi-client-go-tests-in-ci
Jun 24, 2026
Merged

Run duneapi-client-go tests in CI#29
charisra merged 2 commits into
mainfrom
charis/gro-429-run-duneapi-client-go-tests-in-ci

Conversation

@charisra

@charisra charisra commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds a GitHub Actions CI workflow to run unit tests on push and PR.

Test plan

  • CI passes on the PR
  • E2E tests remain skipped (-short; need DUNE_API_KEY)

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@charisra charisra requested a review from a team June 24, 2026 13:23
@charisra

Copy link
Copy Markdown
Contributor Author

bugbot run

@cursor

cursor Bot commented Jun 24, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
CI-only change with no application or runtime code modified.

Overview
Adds a GitHub Actions workflow that runs on every push and pull_request.

The job checks out the repo, installs Go from go.mod, and runs go test -short -timeout=30s -race -cover ./... on ubuntu-latest.

Reviewed by Cursor Bugbot for commit c19cc72. Configure here.

Comment thread .github/workflows/ci.yml

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit c19cc72. Configure here.

@charisra charisra marked this pull request as ready for review June 24, 2026 13:28
@charisra charisra merged commit bca154a into main Jun 24, 2026
1 check passed
@charisra charisra deleted the charis/gro-429-run-duneapi-client-go-tests-in-ci branch June 24, 2026 13:28
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