-
-
Notifications
You must be signed in to change notification settings - Fork 956
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
test: test case for issue #1120 #1121
Conversation
@szmarczak @sindresorhus please comment this (very) kludgy fix. In short, There is very likely a cleaner way to handle this issue. |
@szmarczak here is a cleaner fix (type casting is somewhat ugly though) |
If you don't mind waiting until I finish up the Rewrite PR, it will be much easier to look into this later... |
Sure. |
This can continue now. #1051 was merged. |
Let me now if and how I may assist |
Thanks for the reproducible example, fixed in 99d70df |
Sample test to reproduce issue #1120
Checklist