Skip to content

Add CI workflow to run tests on pull requests#8

Merged
richhankins merged 2 commits intomainfrom
large-finch-6fhj
Jan 29, 2026
Merged

Add CI workflow to run tests on pull requests#8
richhankins merged 2 commits intomainfrom
large-finch-6fhj

Conversation

@richhankins
Copy link
Copy Markdown
Contributor

Summary

Adds a GitHub Actions CI workflow that runs unit tests on pull requests and pushes to main.

Changes

  • New .github/workflows/ci.yml workflow that:
    • Triggers on PRs to main and pushes to main
    • Uses Node.js 20 with npm caching for faster runs
    • Builds the project (catches TypeScript type errors)
    • Runs the full vitest test suite

Testing

Ran tests locally: 17 test files, 162 tests passed, 24 skipped (integration tests requiring API credentials).

Next Steps

After merging, consider enabling branch protection rules to require the test job to pass before merging PRs.


Pull Request opened by Augment Code with guidance from the PR author

- Run tests on PRs to main and pushes to main
- Uses Node.js 20 with npm caching for faster runs
- Builds project and runs vitest test suite

Agent-Id: agent-197d346e-17e3-4016-8f9a-c7d77e060a1e
@augment-app-staging
Copy link
Copy Markdown

🤖 Augment PR Summary

Summary: Adds a GitHub Actions CI workflow to automatically build and run unit tests on pull requests targeting main and on pushes to main.
Changes: Introduces .github/workflows/ci.yml using Node.js 20 with npm caching, running npm ci, npm run build, and npm test.

🤖 Was this summary useful? React with 👍 or 👎

Copy link
Copy Markdown

@augment-app-staging augment-app-staging bot left a comment

Choose a reason for hiding this comment

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

Review completed. 2 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.


steps:
- name: Checkout code
uses: actions/checkout@v4
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Consider pinning third-party actions (e.g., actions/checkout) to a full commit SHA instead of a moving tag like @v4 to reduce supply-chain risk.

Other Locations
  • .github/workflows/ci.yml:18

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

push:
branches: [main]

jobs:
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

It may be worth explicitly setting workflow/job permissions (e.g., read-only) so the default GITHUB_TOKEN scope is minimized for this CI run.

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

@richhankins richhankins requested a review from igor0 January 29, 2026 01:53
@richhankins richhankins merged commit cd9b6e5 into main Jan 29, 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.

1 participant