Inversifyjs: Error handling: loosing stacktrace

[jsamr]

In the factoryWrapper function, an error is thrown with caught error.message, loosing stacktrace information.
I suggest instead re-throwing the error.

replace (src/syntax/binding_to_syntax.ts#L24)

 throw new Error(error.message);

with (src/syntax/binding_to_syntax.ts#L24)

 throw error:

[remojansen]

Hi @sveinburne thanks for reporting this issue. Would you like to send a PR?

[jsamr]

@remojansen I have spotted other occurrences of this pattern (throw new Error(error.message)).
Are you OK that I dig further and replace'em all in a PR ?

[remojansen]

Yes that should be OK :+1:

[remojansen]

@sveinburne I just released 4.5.2 🎉

[jsamr]

@remojansen Thanks a lot for your reactivity !