nrwl / nx

Smart Monorepos · Fast CI
https://nx.dev
MIT License
23.75k stars 2.37k forks source link

fix(core): reduce time it takes to require nx commands #28884

Closed FrozenPandaz closed 1 week ago

FrozenPandaz commented 1 week ago

Current Behavior

nx/src/command-line/nx-commands takes ~88-91ms on my machine to require which needs to happen for every single Nx command.

Expected Behavior

nx/src/command-line/nx-commands takes ~30-33ms on my machine to require.

Related Issue(s)

Fixes #

vercel[bot] commented 1 week ago

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

1 Skipped Deployment | Name | Status | Preview | Updated (UTC) | | :--- | :----- | :------ | :------ | | **nx-dev** | ⬜️ Ignored ([Inspect](https://vercel.com/nrwl/nx-dev/at4q7o8MfUzxm8FaARcBm5MVtJy7)) | [Visit Preview](https://nx-dev-git-perf-nrwl.vercel.app) | Nov 11, 2024 8:56pm |
nx-cloud[bot] commented 1 week ago

☁️ Nx Cloud Report

CI is running/has finished running commands for commit fd48dd6bc75872d271262c09e3a23a2e95522dff. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 4 targets - [`nx affected --targets=lint,test,build,e2e,e2e-ci --base=19b0828d278b5fa8e3d9f8fc537b317c7f442848 --head=fd48dd6bc75872d271262c09e3a23a2e95522dff --parallel=3`](https://staging.nx.app/runs/ZHUrHRzaIA?utm_source=pull-request&utm_medium=comment) - [`nx run-many -t check-imports check-commit check-lock-files check-codeowners documentation --parallel=1 --no-dte`](https://staging.nx.app/runs/q3YsRjwl1w?utm_source=pull-request&utm_medium=comment) - [`nx-cloud record -- nx format:check --base=19b0828d278b5fa8e3d9f8fc537b317c7f442848 --head=fd48dd6bc75872d271262c09e3a23a2e95522dff`](https://staging.nx.app/runs/W8Bkj12QGG?utm_source=pull-request&utm_medium=comment) - [`nx documentation --no-dte`](https://staging.nx.app/runs/ViUFLCnlez?utm_source=pull-request&utm_medium=comment)

Sent with 💌 from NxCloud.

github-actions[bot] commented 5 days ago

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.