Skip to content

Add CI workflow - #44

Merged
johannesschiessl merged 3 commits into
mainfrom
t3code/add-github-ci-workflow
Aug 4, 2026
Merged

Add CI workflow#44
johannesschiessl merged 3 commits into
mainfrom
t3code/add-github-ci-workflow

Conversation

@johannesschiessl

Copy link
Copy Markdown
Member

Summary

  • Add GitHub Actions CI for pushes to main, pull requests, and manual runs.
  • Verify formatting, linting, types, and tests with pinned Node.js and pnpm versions.
  • Build the web and desktop applications with concurrency cancellation and a 20-minute timeout.

Testing

  • Not run locally; this change adds the CI checks that run in GitHub Actions.

- Check formatting, lint, types, and tests
- Build the web and desktop applications

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 1 file

Architecture diagram
sequenceDiagram
    participant GitHub as GitHub (Push/PR/Manual)
    participant Actions as GitHub Actions CI
    participant Runner as Verify Job (ubuntu-latest)
    participant Repo as Repository
    participant Pnpm as pnpm 11.9.0
    participant Node as Node.js 24
    participant Code as Codebase

    Note over GitHub,Code: CI Pipeline Flow

    alt Event: Push to main OR Pull request OR Manual trigger
        GitHub->>Actions: Trigger workflow
    end

    Actions->>Actions: Set concurrency: ci-$workflow-$ref
    opt If previous run in progress
        Actions->>Actions: Cancel in-progress run
    end

    Actions->>Runner: Start verify job (timeout: 20 min)
    Runner->>Repo: Check out code
    Repo-->>Runner: Repository code

    Runner->>Pnpm: Set up pnpm (version 11.9.0)
    Pnpm-->>Runner: pnpm ready

    Runner->>Node: Set up Node.js (version 24) with pnpm cache
    Node-->>Runner: Node.js ready

    Runner->>Pnpm: pnpm install --frozen-lockfile
    Pnpm->>Code: Install dependencies from lockfile
    Code-->>Pnpm: Dependencies installed
    Pnpm-->>Runner: Success

    Runner->>Pnpm: pnpm exec vp check
    Pnpm->>Code: Formatting, lint, type check
    Code-->>Pnpm: Results
    Pnpm-->>Runner: Pass/fail

    Runner->>Pnpm: pnpm exec vp run -r test
    Pnpm->>Code: Run all tests
    Code-->>Pnpm: Test results
    Pnpm-->>Runner: Pass/fail

    Runner->>Pnpm: pnpm --filter @showtime/web build
    Pnpm->>Code: Build web app
    Code-->>Pnpm: Build output
    Pnpm-->>Runner: Success/failure

    Runner->>Pnpm: pnpm --filter @showtime/desktop exec tsc
    Pnpm->>Code: TypeScript compilation for desktop
    Code-->>Pnpm: Compiled code
    Pnpm-->>Runner: Success/failure

    Runner->>Pnpm: pnpm --filter @showtime/desktop exec vp build --config electron.vite.config.ts
    Pnpm->>Code: Build desktop app with Electron
    Code-->>Pnpm: Build output
    Pnpm-->>Runner: Success/failure

    alt All steps succeed
        Runner-->>Actions: Job passed
        Actions-->>GitHub: ✅ Green check
    else Any step fails
        Runner-->>Actions: Job failed
        Actions-->>GitHub: ❌ Red X
    end
Loading

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread .github/workflows/ci.yml
@johannesschiessl johannesschiessl changed the title Add GitHub CI workflow Add CI workflow Aug 4, 2026
- Remove the redundant standalone web build step
- Run the desktop package build command
@johannesschiessl
johannesschiessl merged commit 52bf048 into main Aug 4, 2026
2 checks passed
@johannesschiessl
johannesschiessl deleted the t3code/add-github-ci-workflow branch August 4, 2026 17: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.

1 participant