clone does not clone error instance

[5d-jh]

const target =new Error();
const cloned = clone(target);

// actual
target === cloned // true

//expected
target === cloned // false

[kedashoe]

Hello @5d-jh ! Ya, we do not attempt to handle Error: https://github.com/ramda/ramda/blob/afe98b03c322fc4d22742869799c9f2796c79744/source/internal/_clone.js#L40

Looks like lodash just returns a plain Object with some properties set. Similar with a couple other deep clone type packages I saw.

Seems to me like creating a new Error with some properties set on it would be better? Would you like to submit a PR?

[5d-jh]

Sure. I will handle this issue

[richecr]

Hi everyone, how is this issue progressing?

I wanted to start contributing to the project, can I start with this issue?

[kedashoe]

@richecr sounds good, go for it!