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

Update Location of typescript-import-locator #287

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Speuce
Copy link

@Speuce Speuce commented Apr 2, 2023

Current Behavior

When running npm i on a project with nx >= 15.9.0:

Failed to patch Nx dep-graph for Vue support. Error: Could not find Nx's dep-graph builder in node_modules
    at getFilePath (/Users/matt/Documents/GitHub/appbuilder/node_modules/@nx-plus/vue/patch-nx-dep-graph.js:39:9)
    at patchNxDepGraph (/Users/matt/Documents/GitHub/appbuilder/node_modules/@nx-plus/vue/patch-nx-dep-graph.js:10:22)
    at Object.<anonymous> (/Users/matt/Documents/GitHub/appbuilder/node_modules/@nx-plus/vue/patch-nx-dep-graph.js:42:1)
    at Module._compile (node:internal/modules/cjs/loader:1275:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1329:10)
    at Module.load (node:internal/modules/cjs/loader:1133:32)
    at Module._load (node:internal/modules/cjs/loader:972:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:83:12)
    at node:internal/main/run_main_module:23:47

Expected Behavior

Successfully patched Nx dep-graph for Vue support.

Related Issue(s)

Fixes #286

Brief Description

This Commit in the nx project moved typescript-import-locator to another location. This PR adds that new location to the possiblePaths list in patch-nx-graph.js.

@gionkunz
Copy link

@ZachJW34, can you please check this PR?

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.

Nx moving JS graph building to new location
3 participants