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): use package manager workspace globs to find projects #9131
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/nrwl/nx-dev/5GkuqjWaWRNHgnpHHeMusP44VcXF [Deployment for 80bc323 canceled] |
f29b964
to
b27c1cb
Compare
} | ||
|
||
function jsPluginConfig(nxJson: NxJsonConfiguration): NrwlJsPluginConfig { | ||
return nxJson?.pluginsConfig['@nrwl/js'] ?? {}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
return nxJson?.pluginsConfig['@nrwl/js'] ?? {}; | |
return nxJson?.pluginsConfig?.['@nrwl/js'] ?? {}; |
b27c1cb
to
38c3fa5
Compare
38c3fa5
to
35a0ef7
Compare
packages/tao/src/shared/workspace.ts
Outdated
@@ -784,6 +820,8 @@ export function buildWorkspaceConfigurationFromGlobs( | |||
): WorkspaceJsonConfiguration { | |||
const projects: Record<string, ProjectConfiguration> = {}; | |||
|
|||
console.log({ projectFiles }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make sure to remove this, or lock it behind NX_VERBOSE_LOGGING one
35a0ef7
to
80bc323
Compare
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. |
Current Behavior
Every single package.json file is used to determine projects for the workspace when
workspace.json
does not exist.Expected Behavior
Package manager workspaces glob patterns are used to find projects.
Pnpm is not supported yet.
Related Issue(s)
Fixes #