Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Aug 24, 2025

This PR adds a comprehensive CI workflow that automatically tests, lints, and builds the vibe-dashboard project on every pull request and push to main/develop branches.

What's Added

A new CI workflow (.github/workflows/ci.yml) that:

  • Runs on multiple triggers: Pull requests and pushes to main and develop branches
  • Uses Node.js LTS: Ensures compatibility and stability with the latest Long Term Support version
  • Uses efficient caching: pnpm store caching for faster subsequent runs
  • Executes all quality checks:
    • pnpm lint - ESLint validation across all packages
    • pnpm test - Test execution (currently no tests configured, but ready for future)
    • pnpm build - TypeScript compilation and Vite production build
  • Verifies build artifacts: Ensures the build process creates expected output files

Why This Matters

Previously, the repository only had a deployment workflow that runs on the main branch. This new CI workflow provides:

  • Early feedback during development via pull request checks
  • Quality assurance before code is merged
  • Stable environment using Node.js LTS for consistent builds
  • Consistent environment using the same tooling as production deployment

The workflow is designed to complement the existing deploy.yml workflow without conflicts, providing comprehensive automated testing for the development workflow.

Testing

The CI workflow has been validated to:

  • ✅ Successfully run all commands in a clean environment
  • ✅ Catch and fail on linting errors (tested with intentional ESLint violations)
  • ✅ Catch and fail on TypeScript compilation errors
  • ✅ Verify build artifacts are properly created
  • ✅ Use efficient caching to minimize build times

Fixes #19.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Add a CI workflow to test, lint and build Add CI workflow to test, lint and build Aug 24, 2025
Copilot AI requested a review from Boshen August 24, 2025 12:08
test-lint-build:
runs-on: ubuntu-latest

strategy:
Copy link
Member

Choose a reason for hiding this comment

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

@copilot remove this strategy

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed the strategy matrix section as requested. The CI now runs on a single job instead of multiple Node.js versions. (20ec90d)

- name: Setup Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node-version }}
Copy link
Member

Choose a reason for hiding this comment

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

@copilot use node lts

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changed to use Node.js LTS (lts/*) instead of specific versions. This provides better stability and automatically uses the latest LTS version. (20ec90d)

Copilot AI requested a review from Boshen August 24, 2025 14:31
@Boshen Boshen marked this pull request as ready for review August 24, 2025 14:49
@Boshen Boshen merged commit e188a52 into main Aug 24, 2025
1 check passed
@Boshen Boshen deleted the copilot/fix-19 branch August 24, 2025 14:49
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.

Add a CI workflow to test, lint and build

2 participants