Coverage increased (+0.0007%) to 97.689% when pulling 5d6291e on martinsik:add-index-to-delaywhen into 046b03d on ReactiveX:master.

Change looks fine, but I assume TypeScript users who used selector before encounter compilation error by selector signature changes?

@kwonoj I don't think so... since arguments are optional in JavaScript, TypeScript shouldn't complain I don't think.

@kwonoj confirmed... (a: A, b: B) => void matches (a: A) => void just fine in TS.

Thanks for confirmation. I'll check this in around today to tomorrow.

Oops.... @kwonoj, we didn't want to merge this new feature until after we patched the existing minor.

@benlesh 😱 should this be reverted?

@kwonoj yes, please. What I'd like to do is merge all of the minor changes after I merge all of the patches. So we can publish 5.2.1 and 5.3.0 at the same time.

ugh.. so we're going to have to re-merge this later. :) @kwonoj can you open another PR with these changes? (no worries @martinsik, your github credit is already in the history :) )

Yup, will do.

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.