-
Notifications
You must be signed in to change notification settings - Fork 12.9k
Fix return value and error reporting for getIterationTypesOfMethod #50146
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
Thanks for fixing this, I think now the iterator/iterable functions will make more sense. |
@typescript-bot test this |
Heya @rbuckton, I've started to run the diff-based user code test suite on this PR at 51be560. You can monitor the build here. Update: The results are in! |
@rbuckton Here they are:Comparison Report - main..refs/pull/50146/merge [adonis-framework]1 of 1 projects failed to build with the old tsc /mnt/ts_downloads/adonis-framework/tsconfig.json
|
Not actually a new error, this previously said |
51be560
to
98507a1
Compare
This fixes
getIterationTypesOfMethod
to correctly returnundefined
when the iterator type does not have a validnext
method. This also ensures related errors are still reported when we've already cached anoIterationTypes
result.Related #49736