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

Potential changes required for typescript 3.5.0 #939

Closed
konpikwastaken opened this issue May 24, 2019 · 4 comments · Fixed by #942
Closed

Potential changes required for typescript 3.5.0 #939

konpikwastaken opened this issue May 24, 2019 · 4 comments · Fixed by #942

Comments

@konpikwastaken
Copy link
Contributor

Expected Behaviour

  • Specify incremental in tsconfig.json file
  • Project is able to compile successfully with against typescript@3.5.0

Note that this issue is not present with typescript@3.4.x

Actual Behaviour

An error similar to the following one is received:

[tsl] ERROR
      TS5074: Option '--incremental' can only be specified using tsconfig, emitting to single file or when option `--tsBuildInfoFile` is specified

Per conversation with typescript team (microsoft/TypeScript/issues/31447), it appears that this is a change in typescript but a change in ts-loader will be required.

Current workaround is that users will also need to specify tsBuildInfo in their package.json (whereas they previously did not).

Location of a Minimal Repository that Demonstrates the Issue.

https://github.com/konpikwastaken/broken-example-ts-loader-tsc

@johnnyreilly
Copy link
Member

Will this be handled by #935 ?

@konpikwastaken
Copy link
Contributor Author

I don't believe the two are related. #935 is specific to compiling project references using the solution builder. The issue raised here has to do with usage of incremental flag set via tsconfig.

That's the issue with ts-loader where in it doesn't indicate that compiler options are from config file. if you specify tsbuildinfo file path in your config file the error should go away
(microsoft/TypeScript#31447 (comment))

@johnnyreilly
Copy link
Member

Gotcha. Yes - makes sense now I think about it. If you want to have a go at adding what's needed that'd be amazing. Don't worry if you need to ask questions along the way and tests don't pass - that's all totally cool and something we can collaborate on.

@konpikwastaken
Copy link
Contributor Author

@johnnyreilly PR submitted :)

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 a pull request may close this issue.

2 participants