hapijs / hoek

Node utilities shared among the extended hapi universe
Other
480 stars 171 forks source link

Clone Map and Set #297

Closed kanongil closed 5 years ago

kanongil commented 5 years ago

This is a continuation of #248, containing expanded and revised tests.

I decided to remove the Promise cloning altogether after some thought. The reason is that, with node native promises, it is impossible to do a clone without potentially changing the behavior of the source Promise. More specifically, attaching a catch handler means that it can no longer trigger an unhandledRejection process error.

lock[bot] commented 4 years ago

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.