-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Added .d.ts
files for test utils
#2616
Conversation
@dotansimha All the fixes in existing |
Co-authored-by: Ivan Goncharov <ivan.goncharov.ua@gmail.com>
@IvanGoncharov I saw you already moved the fixes to a new PR and merged, so that's great! Thank you. Do you think I should rebase #2609 to this PR? because those changes are needed in order to support the TS tests. |
This can be closed considering that the code base has been converted to TS. Thank you @dotansimha for your work on this, which has been integrated into other PRs as well as for your energy and initiative in pushing forward the migration. |
Originally from: #2609 , those fixes are required in order to make the tests compatible with TS.