Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Jun 12, 2017

Fixes #16452

@rbuckton I wanted to test this with for-await-of but couldn't seem to get it to work.
I tried:

// @lib: esnext

////...existing test...
////async function f(points: AsyncIterable<{ x: number, y: number }>) {
////    for await (const { /*d*/ } of points) {}
////}
verify.noErrors();

I get: minChar: 173, limChar: 186, message: Cannot find name 'AsyncIterable'.

@ghost ghost force-pushed the completionsDestructuring branch from 298cf2c to 0646d84 Compare June 12, 2017 16:17
@mhegazy
Copy link
Contributor

mhegazy commented Jun 12, 2017

@Andy-MS the error indicates the lib is not loaded correctly. is this a test-only issue or you see it with VSCode as well?

@ghost
Copy link
Author

ghost commented Jun 12, 2017

Test-only.

@mhegazy
Copy link
Contributor

mhegazy commented Jun 12, 2017

We need to look into why fourslash is not including the library correctly.

@ghost ghost mentioned this pull request Jun 12, 2017
@ghost ghost merged commit 7fb821e into master Jun 12, 2017
@ghost ghost deleted the completionsDestructuring branch June 12, 2017 19:12
@microsoft microsoft locked and limited conversation to collaborators Jun 14, 2018
This pull request was closed.
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