Skip to content

chore(ci): add build, lint, and test workflow#41

Merged
bnema merged 1 commit into
mainfrom
improve/001-ci-pipeline
Jul 8, 2026
Merged

chore(ci): add build, lint, and test workflow#41
bnema merged 1 commit into
mainfrom
improve/001-ci-pipeline

Conversation

@bnema

@bnema bnema commented Jul 8, 2026

Copy link
Copy Markdown
Owner

Summary

  • add GitHub Actions CI for build, lint, and race tests
  • pin workflow actions and goimports, add timeout and concurrency controls

Verification

  • go build -o vev .
  • make lint
  • make test
  • CodeRabbit CLI: 0 findings

Stack: 1/6, base branch for the next improve plan.

Summary by CodeRabbit

  • Chores
    • Added an automated CI workflow for pull requests and pushes to the main branch.
    • Runs build, lint, and test checks with read-only workflow permissions.
    • Uses concurrency to cancel superseded runs, plus a 15-minute timeout to keep validation fast and reliable.

Copilot AI review requested due to automatic review settings July 8, 2026 13:50
@coderabbitai

coderabbitai Bot commented Jul 8, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: ba051c98-f899-4903-bbf9-2fc16b40c00d

📥 Commits

Reviewing files that changed from the base of the PR and between e98f630 and fa92ec3.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml

📝 Walkthrough

Walkthrough

Adds a GitHub Actions workflow that runs on pushes to main and pull requests, sets workflow concurrency and read-only permissions, and executes a Go build, lint, and test job.

Changes

CI Workflow

Layer / File(s) Summary
CI workflow definition
.github/workflows/ci.yml
Defines the workflow triggers, concurrency behavior, read-only permissions, and the test job that checks out code, sets up Go, installs goimports, and runs build, lint, and test.

Estimated code review effort: 2 (Simple) | ~10 minutes

Poem

A rabbit hopped through CI tonight,
With builds and tests all running right.
Main branch now hums, pull requests too,
goimports joins the clover dew.
🐇✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the new CI workflow that runs build, lint, and tests.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch improve/001-ci-pipeline

Comment @coderabbitai help to get the list of available commands.

Copilot AI 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.

Pull request overview

Adds a GitHub Actions CI workflow to run core Go build/lint/test checks on PRs and main branch pushes, aligning automated CI with the repository’s existing local verification commands (go build, make lint, make test).

Changes:

  • Introduces a ci workflow that builds the vev binary, runs make lint, and runs make test (race-enabled).
  • Pins actions/checkout and actions/setup-go to specific SHAs and sets minimal permissions (contents: read).
  • Adds concurrency cancelation and a job timeout to reduce redundant CI runs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@coderabbitai coderabbitai 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.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/workflows/ci.yml:
- Around line 21-26: The workflow is still pinned to older major versions of
actions/checkout and actions/setup-go, so update the uses entries in ci.yml to
the current majors and replace the SHA pins with the matching new commit SHAs.
Keep the existing step structure and inputs the same, but refresh the references
for the checkout and setup-go actions so the workflow uses the latest supported
versions.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: a1709997-231a-4b44-82c9-2b2dfc675637

📥 Commits

Reviewing files that changed from the base of the PR and between 91b82aa and e98f630.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml

Comment thread .github/workflows/ci.yml Outdated
@bnema
bnema force-pushed the improve/001-ci-pipeline branch from e98f630 to fa92ec3 Compare July 8, 2026 15:05
@bnema
bnema merged commit ffafa26 into main Jul 8, 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.

2 participants