Skip to content

ci: run @evlog/cli tests and publish it via pkg-pr-new#449

Merged
HugoRCD merged 1 commit into
mainfrom
ci/test-cli-in-pipeline
Jul 24, 2026
Merged

ci: run @evlog/cli tests and publish it via pkg-pr-new#449
HugoRCD merged 1 commit into
mainfrom
ci/test-cli-in-pipeline

Conversation

@HugoRCD

@HugoRCD HugoRCD commented Jul 24, 2026

Copy link
Copy Markdown
Owner

Problem

packages/cli (@evlog/cli) has 15 test files / 84 tests, but nothing in .github/workflows/ ran them:

  • the test job only runs vitest inside packages/evlog
  • the publish (pkg-pr-new) job only published ./packages/evlog, ./packages/nuxthub, ./packages/telemetry

So the CLI had no CI test coverage and no PR preview install link.

Fix

  • add a dedicated cli job that runs pnpm exec turbo run test --filter=@evlog/cli (same pattern as the existing telemetry job)
  • add cli to the publish job's needs
  • add ./packages/cli to the pkg-pr-new publish command

No changeset — internal CI config only, doesn't touch the published package.

Summary by CodeRabbit

  • Tests

    • Added automated testing for the CLI package as part of continuous integration.
    • Publishing now waits for CLI tests to complete successfully.
  • New Features

    • The CLI package is now included in preview package publishing.

@vercel

vercel Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
evlog-docs Ready Ready Preview, Comment, Open in v0 Jul 24, 2026 2:51pm
evlog-telemetry Ready Ready Preview, Comment Jul 24, 2026 2:51pm
just-use-evlog Ready Ready Preview, Comment Jul 24, 2026 2:51pm

Request Review

@github-actions github-actions Bot added the ci label Jul 24, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thank you for following the naming conventions! 🙏

@HugoRCD
HugoRCD merged commit 7ebca51 into main Jul 24, 2026
18 of 19 checks passed
@HugoRCD
HugoRCD deleted the ci/test-cli-in-pipeline branch July 24, 2026 14:53
@coderabbitai

coderabbitai Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: ce801a10-84da-4b49-832c-f0759aec5714

📥 Commits

Reviewing files that changed from the base of the PR and between e6fc16d and 32a5d94.

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

📝 Walkthrough

Walkthrough

The CI workflow adds a dedicated CLI test job, requires it before publishing, and includes ./packages/cli in the published package set.

Changes

CLI CI and publishing

Layer / File(s) Summary
CLI validation and publishing
.github/workflows/ci.yml
Adds a CLI test job, includes it in the publish job dependencies, and publishes the CLI package.

Estimated code review effort: 1 (Trivial) | ~5 minutes

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci/test-cli-in-pipeline

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant