Skip to content

Add warn that user can not use --strictPropertyInitialization without --strictNullChecks #23664

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

Conversation

a-tarasyuk
Copy link
Contributor

Fixes part of #23659

@mhegazy
Copy link
Contributor

mhegazy commented Apr 25, 2018

thanks @a-tarasyuk. we also need a test to go with this change. see tests\cases\compiler\optionsInlineSourceMapMapRoot.ts for an example.

@@ -0,0 +1,70 @@
// @strictPropertyInitialization: true
// @strictNullChecks: true
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry i was not clear. i meant a test for the new error message.. i.e.

// @strictPropertyInitialization: true
// @strictNullChecks: false

var x;

@a-tarasyuk a-tarasyuk force-pushed the feature/strictPropertyInitialization-can-not-be-used-without-strictNullChecks branch from b143af3 to f04a18a Compare April 26, 2018 17:19
@mhegazy mhegazy merged commit 2c956af into microsoft:master Apr 30, 2018
@mhegazy
Copy link
Contributor

mhegazy commented Apr 30, 2018

thanks!

@microsoft microsoft locked and limited conversation to collaborators Jul 31, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants