-
Notifications
You must be signed in to change notification settings - Fork 2
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
doesn't support jsconfig.json #2
Comments
Hi, @Glenn-Sharecare! Pretty sure In my opinion it should not be used to configure how the project is built. I am not totally against having a single file that configures how path aliases for vanilla Javascript projects. But I am certain that it should not be in the same project/package as the TypeScript resolver since it would blur out the responsibility of this package. Be free to fork this project and create a version for |
jsconfig is just a js specific version of tsconfig, which is supported in this library. Here is the typescript page on tsconfig that also covers jsconfig (briefly). https://www.typescriptlang.org/docs/handbook/tsconfig-json.html It is true that VS code supports it, which is nice. Not having to repeat yourself and try to maintain two lists of aliases is pretty swell. In fact it'd be really cool if webpack supported it as well. |
From the brief description that Typescript docs have on it I see Currently it seems like Parcel would not allow its plugins to access any command line arguments (or implement additional ones). I can see 2 options here and both seems like quite poor hacks:
The second option is very dirty and kind of defeats the purpose. Still both options suffer from a giant issue - even if this resolver is able to read Typescript configuration from a different file Parcel can't! From what I found in Parcel source code it always looks for In conclusion, I do not think it makes sense to teach this plugin reading different BTW looking at Parcel sources I found out that it resolves |
It would be a really easy change to support this as well. same format, different name
The text was updated successfully, but these errors were encountered: