Skip to content
This repository was archived by the owner on May 12, 2026. It is now read-only.

chore: remove redundant corepack enable from CD workflow - #20

Merged
oduwoleeyinojuoluwa44 merged 1 commit into
devfrom
chore/ci-cd-configuration
May 10, 2026
Merged

chore: remove redundant corepack enable from CD workflow#20
oduwoleeyinojuoluwa44 merged 1 commit into
devfrom
chore/ci-cd-configuration

Conversation

@leothatguy

@leothatguy leothatguy commented May 10, 2026

Copy link
Copy Markdown
Contributor

Description

Removed the corepack process so i dont get the permission error

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate - Postman, etc):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Summary by CodeRabbit

  • Chores
    • Streamlined the deployment workflow by removing a package manager initialization step while preserving all build and dependency installation functionality.

Review Change Stack

@gemini-code-assist

Copy link
Copy Markdown

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@coderabbitai

coderabbitai Bot commented May 10, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: ed830c0d-dc1b-4c57-b545-9c435d40f2f1

📥 Commits

Reviewing files that changed from the base of the PR and between 80befbd and c0660b7.

📒 Files selected for processing (1)
  • .github/workflows/cd.yml
💤 Files with no reviewable changes (1)
  • .github/workflows/cd.yml

📝 Walkthrough

Walkthrough

The CD workflow's deployment script removes the corepack enable command, allowing the dependency installation (pnpm install --frozen-lockfile) and build step to execute immediately after environment file setup.

Changes

CD Workflow Simplification

Layer / File(s) Summary
Deployment Script
.github/workflows/cd.yml
The corepack enable command is removed from the remote deploy script's "Install dependencies and build" section.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • hngprojects/skillbridge-be#14: Directly related—PR #14 introduces the deploy script with corepack enable, while this PR removes that same initialization step.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: removing the corepack enable command from the CD workflow, which directly matches the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/ci-cd-configuration

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.

@oduwoleeyinojuoluwa44
oduwoleeyinojuoluwa44 merged commit 15574a6 into dev May 10, 2026
2 checks passed
kenneropia pushed a commit that referenced this pull request May 11, 2026
chore: remove redundant corepack enable from CD workflow
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants