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

fix: incremental is default true only if composite is #1110

Merged
merged 1 commit into from
Sep 18, 2020

Conversation

agilgur5
Copy link
Contributor

Description

  • Compiler Options docs specify this behavior as such
    • which I assume means if composite is false, incremental is also
      false unless set explicitly, but I have never used it myself

Tags

Similar to #1102 and #1101 , found while investigating discrepancies between the TSConfig Reference and Compiler Options after finding #1092

Review Notes

I've not actually used this option per the description, so I haven't tested to see which is accurate. Perhaps the TSConfig Reference was correct and the Compiler Options docs were wrong and so this alignment change should be inverted

- Compiler Options docs specify this behavior as such
  - which I assume means if composite is false, incremental is also
    false unless set explicitly, but I have never used it myself
@orta
Copy link
Contributor

orta commented Sep 18, 2020

I read the compiler to try and figure this out, it generally looks like this is true and I think the 3.4 release notes also back up this change - thanks

@orta orta merged commit 0e694e8 into microsoft:v2 Sep 18, 2020
@agilgur5
Copy link
Contributor Author

Thanks for double-checking; yea my read on the 3.4 release notes is similar

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 this pull request may close these issues.

2 participants