Skip to content

feat(ci): reusable cli install test workflow, windows nightly#7953

Merged
paulbalaji merged 3 commits intomainfrom
speed-up-cross-platform-cli-test
Jan 30, 2026
Merged

feat(ci): reusable cli install test workflow, windows nightly#7953
paulbalaji merged 3 commits intomainfrom
speed-up-cross-platform-cli-test

Conversation

@paulbalaji
Copy link
Collaborator

@paulbalaji paulbalaji commented Jan 30, 2026

Summary

  • Extract cross-platform CLI install test to reusable workflow (cli-install-test.yml)
  • Release workflow calls it without Windows (~3 min vs ~13 min)
  • Nightly at 6am UTC runs full matrix including Windows
  • Manual dispatch available with option to include Windows

Timing from original run:

OS Time
Ubuntu ~1m 20s
macOS ~3m
Windows ~13m

Removing Windows from release saves ~10 minutes per release while still catching Windows issues nightly.

Test plan

  • Verify release workflow runs without Windows
  • Verify nightly schedule triggers with Windows
  • Manual dispatch works with include-windows option

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Introduced a centralized CLI installation test workflow to improve CI/CD maintainability
    • Consolidated testing configuration with conditional Windows platform testing during scheduled runs
    • Validates CLI installation across Ubuntu, macOS, and Windows with Node.js versions 24 and 25

✏️ Tip: You can customize this high-level summary in your review settings.

@github-project-automation github-project-automation bot moved this to In Review in Hyperlane Tasks Jan 30, 2026
@paulbalaji paulbalaji force-pushed the speed-up-cross-platform-cli-test branch 6 times, most recently from f517a1f to 744f7ac Compare January 30, 2026 12:45
paulbalaji and others added 2 commits January 30, 2026 12:47
Extract the cross-platform CLI install test to a reusable workflow
(cli-install-test.yml) that can be called with or without Windows.

- Release workflow: calls without Windows (~3 min vs ~13 min)
- Nightly (6am UTC): runs full matrix including Windows
- Manual dispatch: can choose to include Windows

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@paulbalaji paulbalaji force-pushed the speed-up-cross-platform-cli-test branch from 744f7ac to ce75aa6 Compare January 30, 2026 12:48
@paulbalaji paulbalaji changed the title ci: speed up cross-platform CLI install test ci: extract CLI install test to reusable workflow, run Windows nightly Jan 30, 2026
@paulbalaji paulbalaji changed the title ci: extract CLI install test to reusable workflow, run Windows nightly feat(ci): reusable cli install test workflow, windows nightly Jan 30, 2026
@paulbalaji
Copy link
Collaborator Author

@claude review

@paulbalaji paulbalaji marked this pull request as ready for review January 30, 2026 12:59
@claude

This comment was marked as resolved.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 30, 2026

📝 Walkthrough

Walkthrough

This PR introduces a reusable GitHub Actions workflow for testing CLI installation across multiple platforms and Node.js versions, then refactors the release workflow to invoke it. The new cli-install-test.yml workflow conditionally includes Windows runners and consolidates redundant test logic.

Changes

Cohort / File(s) Summary
New CLI Install Test Workflow
.github/workflows/cli-install-test.yml
Introduces reusable workflow with configurable Windows inclusion, testing CLI installation on Ubuntu, macOS, and conditionally Windows using Node.js 24 and 25.
Release Workflow Consolidation
.github/workflows/release.yml
Replaces inline CLI test matrix and steps with centralized workflow call, passing include-windows: false to skip Windows builds during release testing.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

Suggested reviewers

  • ameten
  • xeno097
  • yjamin

Poem

🧅 Some say workflows have layers, some say it's just fluff,
But when you consolidate the testing stuff,
Well that's when things get better, more organized, you see,
One workflow to test 'em all—orderly and free! 🚀

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main changes: introducing a reusable CLI install test workflow with Windows support for nightly runs.
Description check ✅ Passed The description covers the key sections from the template with clear explanations of changes, timing improvements, and a test plan, though some template sections remain incomplete.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch speed-up-cross-platform-cli-test

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 and usage tips.

@paulbalaji paulbalaji enabled auto-merge January 30, 2026 13:18
@paulbalaji paulbalaji added this pull request to the merge queue Jan 30, 2026
Merged via the queue into main with commit 12af0ad Jan 30, 2026
204 of 206 checks passed
@paulbalaji paulbalaji deleted the speed-up-cross-platform-cli-test branch January 30, 2026 13:31
@github-project-automation github-project-automation bot moved this from In Review to Done in Hyperlane Tasks Jan 30, 2026
@codecov
Copy link

codecov bot commented Jan 30, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.02%. Comparing base (0515d2e) to head (ab23504).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7953   +/-   ##
=======================================
  Coverage   77.02%   77.02%           
=======================================
  Files         117      117           
  Lines        2651     2651           
  Branches      244      244           
=======================================
  Hits         2042     2042           
  Misses        593      593           
  Partials       16       16           
Components Coverage Δ
core 87.80% <ø> (ø)
hooks 71.86% <ø> (ø)
isms 81.10% <ø> (ø)
token 86.67% <ø> (ø)
middlewares 84.98% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants