Skip to content

fix(ci): correct dead-code cleanup install cmd#7

Merged
Nadav011 merged 1 commit intomasterfrom
fix/ci-pnpm-dead-code-cleanup
Apr 4, 2026
Merged

fix(ci): correct dead-code cleanup install cmd#7
Nadav011 merged 1 commit intomasterfrom
fix/ci-pnpm-dead-code-cleanup

Conversation

@Nadav011
Copy link
Copy Markdown
Owner

@Nadav011 Nadav011 commented Apr 4, 2026

Replace the broken 'ppnpm' typo with 'pnpm' in the dead-code cleanup workflow.

Summary by CodeRabbit

  • Chores
    • Fixed build workflow configuration to ensure proper dependency installation.

Copilot AI review requested due to automatic review settings April 4, 2026 11:33
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 4, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 87e91560-6c2c-470d-a592-ac40e20907be

📥 Commits

Reviewing files that changed from the base of the PR and between 5158ae5 and 8ea1b01.

📒 Files selected for processing (1)
  • .github/workflows/dead-code-cleanup.yml

📝 Walkthrough

Walkthrough

A typo correction in the GitHub Actions workflow replaces ppnpm with pnpm in the package installation command for the dead-code cleanup workflow, restoring the correct package manager invocation.

Changes

Cohort / File(s) Summary
Workflow Command Fix
.github/workflows/dead-code-cleanup.yml
Corrected typo in pnpm installation command from ppnpm install --frozen-lockfile to pnpm install --frozen-lockfile.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A typo crept in with an extra 'p',
Double letters where one should be,
But now it's fixed, oh what relief!
The package manager runs with belief,
One keystroke saved from confusion deep. ✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/ci-pnpm-dead-code-cleanup

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.

@Nadav011 Nadav011 merged commit 9c99901 into master Apr 4, 2026
7 of 10 checks passed
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Fixes a typo in the dead-code cleanup GitHub Actions workflow so dependency installation uses the correct pnpm command, restoring the workflow’s ability to run on schedule / dispatch.

Changes:

  • Replace ppnpm install --frozen-lockfile with pnpm install --frozen-lockfile in the dead-code cleanup workflow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants