-
-
Notifications
You must be signed in to change notification settings - Fork 431
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
Make TypeScript basePath
configurable, fixes #619
#621
Conversation
Great! I'll look to include this in the next release. Thanks! |
This seems to break the default tsconfig.json as distributed with create-react-app-typescript. I'm seeing failures with
|
The quick and dirty workaround is to change rootDir to '.' rather than 'src', but I'm not sure whether create react app's tsconfig is wrong, or some other part of the build pipeline is causing the issue. I should really give more info, the default project structure with create-react-app-typescript has tsconfig in the project base path, and src contains all source files, but with this latest change the typescript compiler is looking in src/src for some reason. |
That's weird. As far as I can tell, the provided I'm probably going to have to look into how exactly TypeScript handles the |
If you could look into this I'd appreciate that @loilo. @pettyalex let's see what @loilo comes up with. If all else fails we can revert this change and push out a new version. For now I would pin your version to 2.3.4 though. |
I think that's currently the best solution. I created a quick repro repo where I couldn't nail it down yet, gonna get to work now but going to look at the problem from there. I suggest to move every further communication over to wmonk/create-react-app-typescript#156. |
Enables the behaviour suggested in #619 to modify the
basePath
provided to the TS compiler.Added: