deepmerge icon indicating copy to clipboard operation
deepmerge copied to clipboard

Drop ES5

Open TehShrike opened this issue 5 years ago • 5 comments

I'm ready to convert to modern JS.

A note should probably be added to the readme.

TehShrike avatar Sep 09 '19 22:09 TehShrike

Will also need to note the need for an Object.assign polyfill

TehShrike avatar Oct 21 '19 15:10 TehShrike

Will you use it in mutable or immutable way?

TrySound avatar Oct 21 '19 16:10 TrySound

use what? Object.assign? Object.assign is only used on the options argument. See #166

TehShrike avatar Oct 21 '19 16:10 TehShrike

Maybe use object rest operator and rely on babel preset env output?

TrySound avatar Oct 21 '19 17:10 TrySound

Would you be interested in converting the project to TypeScript?

Would be glad to make a PR.

ghepesdoru avatar Mar 22 '20 17:03 ghepesdoru