Skip to content

Conversation

@cdata
Copy link
Contributor

@cdata cdata commented Sep 18, 2019

Unfortunately, we must revert this part of #747 because it is not compatible with generating external declaration files.

See also microsoft/TypeScript#30355 (comment)

@cdata cdata requested a review from elalish September 18, 2019 00:25
"experimentalDecorators": true,
"noErrorTruncation": true
"noErrorTruncation": true,
"declaration": true
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This should have been enabled previously. Now that it is on, we will see the category of errors this inference pattern raised in google3.

Copy link
Contributor

@elalish elalish left a comment

Choose a reason for hiding this comment

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

Oh well, fun while it lasted.

@cdata cdata merged commit b4cc4c5 into master Sep 18, 2019
@cdata cdata deleted the revert-fancy-mixin-type-inference branch September 18, 2019 16:18
elalish pushed a commit to elalish/model-viewer that referenced this pull request Feb 4, 2020
 
Revert mixin type inference (google#761)
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