chore: remove pnpm from CI workflow package managers#316
Conversation
|
Caution Review failedThe pull request is closed. WalkthroughThe CI workflows were updated to remove pnpm from job matrices. In generate-package.yml and test-nitro-cli.yml, matrices now include only bun and yarn. pnpm-specific setup branches become unreachable, reducing total job combinations and keeping install/build/test paths for bun and yarn intact. Changes
Sequence Diagram(s)sequenceDiagram
participant Dev as GitHub Actions
participant Matrix as Matrix (pm, package-type)
participant Runner as Runner
participant Tool as pm (bun|yarn)
Dev->>Matrix: Expand jobs (pm ∈ {bun,yarn})
Matrix->>Runner: Start job
Runner->>Tool: Setup package manager
Runner->>Tool: Install dependencies
Runner->>Tool: Build / Test
Tool-->>Runner: Exit status
Runner-->>Dev: Report results
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 💡 Knowledge Base configuration:
You can enable these sources in your CodeRabbit configuration. 📒 Files selected for processing (2)
✨ Finishing Touches🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
Summary by CodeRabbit