-
Notifications
You must be signed in to change notification settings - Fork 6
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
References in Nuxt setup #117
Conversation
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.
Could you also resolve the conflicts? Recently there's been a change to that file which did a similar thing, I wonder if your changes overlap with it or not
Got it, yes, let me resolve the conflicts first. Then I'll fix the function and also now I see some other flaws of my solution |
… of where it runs
I fixed the first issue that we discussed in the morning by always transforming Regarding the 2nd problem https://stackblitz.com/edit/nuxt-starter-ap1c4y?file=README.md How to reproduce When When Don't forget to wrap/unwrap the code inside You can temporarily change fileExists, directoryExists parameters from these: to these (I made that change locally for insignificant-slice)Then you'll be able to see how it goes through different possible options (like |
resolveDependency from |
Resolves #94
@illright
The cause of the issue was not exactly in the Nuxt setup. The problem was that
compilerOption.paths
specified in config extensions ("extends": "./.nuxt/tsconfig.json"
) were not resolved and thus path aliases like@/
,~/
didn't work.So such paths from extensions were applied to the FSD root.
I made changes to transform such paths from
../*
to/*
and then apply them to the root.Nuxt setup extends a default config from the
.nuxt
folder it helped us to uncover this issue.Sorry for the delay!