Always write tsbuildInfo when running tsc -b - #58626
Conversation
7dc0b38 to
49286d6
Compare
7356020 to
d8b4a76
Compare
|
TypeScript Bot (@typescript-bot) test it |
|
Hey Sheetal Nandi (@sheetalkamat), the results of running the DT tests are ready. Everything looks the same! |
|
Sheetal Nandi (@sheetalkamat) Here are the results of running the user tests with tsc comparing Everything looks good! |
|
Sheetal Nandi (@sheetalkamat) Here they are:
tscComparison Report - baseline..pr
System info unknown
Hosts
Scenarios
Developer Information: |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Sheetal Nandi (@sheetalkamat) Here are the results of running the top 400 repos with tsc comparing Everything looks good! |
d0723b8 to
e15f648
Compare
e15f648 to
0b0886f
Compare
| commonOptionsWithBuild.forEach(option => { | ||
| if (hasProperty(buildOptions, option.name)) result[option.name] = buildOptions[option.name]; | ||
| }); | ||
| result.tscBuild = true; |
There was a problem hiding this comment.
How will this flag affect people using our APIs to implement build mode like tsc? Will it be set for them? Or can we just assume it's always true?
There was a problem hiding this comment.
If they use our SolutionBuilder API this where we set it so it should work.
There was a problem hiding this comment.
Alrighty; just verifying so people are not surprised.
| commonOptionsWithBuild.forEach(option => { | ||
| if (hasProperty(buildOptions, option.name)) result[option.name] = buildOptions[option.name]; | ||
| }); | ||
| result.tscBuild = true; |
There was a problem hiding this comment.
Alrighty; just verifying so people are not surprised.
For now it only contains
versionbut with plan to add more info about program for tsc -b uptodateness.In future this can be used to:
compositemust list all files (probably)tsc -bnoCheckon commandLine