Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(core): expand meaning of plugins array to allow installing non-runtime plugins and inter-opt with installations property #22108

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

AgentEnder
Copy link
Member

@AgentEnder AgentEnder commented Mar 1, 2024

Current Behavior

Folks using the nx wrapper (nxw.js in .nx) install plugins in nx.json[installation][plugins].

This is a bit weird since there is also nx.json[plugins]

Expected Behavior

You can specify plugins in the top level plugins array.

This PR also adds a field called "runtime" that can be set to false to tell Nx that a plugin worker doesn't need to be spawned for a listed plugin.

This PR fixes up:

  • nx migrate support for plugins array contained versions
  • nx add, to add plugins to the plugins array by default
  • Plugin workers shutdown automatically if a plugin has no runtime functions

Related Issue(s)

Fixes #

@AgentEnder AgentEnder requested review from a team as code owners March 1, 2024 20:55
Copy link

vercel bot commented Mar 1, 2024

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

1 Skipped Deployment
Name Status Preview Updated (UTC)
nx-dev ⬜️ Ignored (Inspect) Visit Preview Jan 15, 2025 6:16pm

Copy link

nx-cloud bot commented Mar 1, 2024

View your CI Pipeline Execution ↗ for commit c831d97.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ❌ Failed 1h 52m 35s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 1m View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx format:check --base=7ef4f... ✅ Succeeded 30s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 28s View ↗
nx documentation --no-dte ✅ Succeeded 1m 10s View ↗

☁️ Nx Cloud last updated this comment at 2025-01-15 20:15:18 UTC

@AgentEnder AgentEnder force-pushed the feat/allow-passing-version-in-plugins-array branch from 87174d3 to 6abd9ed Compare March 1, 2024 22:44
@AgentEnder AgentEnder force-pushed the feat/allow-passing-version-in-plugins-array branch from a4da69e to 238bd08 Compare March 5, 2024 17:25
@AgentEnder AgentEnder force-pushed the feat/allow-passing-version-in-plugins-array branch from 238bd08 to 669083d Compare March 5, 2024 22:18
@AgentEnder AgentEnder force-pushed the feat/allow-passing-version-in-plugins-array branch from 669083d to 3d6678a Compare March 5, 2024 23:19
@AgentEnder AgentEnder force-pushed the feat/allow-passing-version-in-plugins-array branch from 3d6678a to bf8527e Compare April 11, 2024 16:18
@AgentEnder AgentEnder force-pushed the feat/allow-passing-version-in-plugins-array branch from bf8527e to c474c2c Compare May 1, 2024 17:43
@AgentEnder AgentEnder force-pushed the feat/allow-passing-version-in-plugins-array branch from c474c2c to 77c872c Compare July 24, 2024 16:39
@AgentEnder AgentEnder force-pushed the feat/allow-passing-version-in-plugins-array branch from 77c872c to c831d97 Compare January 15, 2025 18:14
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.

1 participant