Skip to content
This repository has been archived by the owner on Nov 5, 2024. It is now read-only.

fix 🐛: (cli) Don't install optional Peer Dependencies #118

Merged
merged 1 commit into from
Oct 20, 2022

Conversation

nachoaldamav
Copy link
Owner

Ultra was installing optional Peer Dependencies like node-sass that takes a lot of time to run post install scripts.

Ultra was installing optional Peer Dependencies like `node-sass` that takes a lot of time to run post install scripts.
@vercel
Copy link

vercel bot commented Oct 20, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
fnpm ✅ Ready (Inspect) Visit Preview Oct 20, 2022 at 7:44PM (UTC)

@nachoaldamav nachoaldamav merged commit b3f04d3 into main Oct 20, 2022
@github-actions
Copy link

Benchmark results:

  • OS: ubuntu-latest
  • Node.js: 18
  • Commit: b3f04d3
  • Pull request title: fix 🐛: (cli) Don't install optional Peer Dependencies
Name Time Group
Bun install (with cache / with lockfile) 0.62s 3
⚡ ULTRA install (with cache / with lockfile) 2.53s 3
⚡ ULTRA install (with cache / no lockfile) 3.56s 2
PNPM install (with cache / with lockfile) 6.25s 3
Bun install (no cache / no lockfile) 7.37s 1
Bun install (with cache / no lockfile) 7.47s 2
PNPM install (with cache / no lockfile) 2
YARN install (with cache / with lockfile) 13.45s 3
NPM install (with cache / with lockfile) 17.85s 3
NPM install (with cache / no lockfile) 25.72s 2
PNPM install (no cache / no lockfile) 1
YARN install (with cache / no lockfile) 35.72s 2
⚡ ULTRA install (no cache / no lockfile) 35.96s 1
YARN install (no cache / no lockfile) 53.07s 1
NPM install (no cache / no lockfile) 56.88s 1

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.

1 participant