Add CommonJS compatibility

[astolcenburg]

With this change, the module can be directly included
by node.js applications that use the CommonJS module system.

[CLAassistant]

All committers have signed the CLA.

[dersanli]

With this change, the module can be directly included
by node.js applications that use the CommonJS module system.

Hi @astolcenburg, currently I can't include ThunderJS like:
import ThunderJS from 'ThunderJS';
Does your PR solve this issue?

[astolcenburg]

With this change, the module can be directly included
by node.js applications that use the CommonJS module system.

Hi @astolcenburg, currently I can't include ThunderJS like:
import ThunderJS from 'ThunderJS';
Does your PR solve this issue?

Hi @dersanli ,

yes, it will work thanks to the interoperability of ECMAScript modules with CommonJS: https://nodejs.org/api/esm.html#esm_interoperability_with_commonjs . But you need to use node.js >= 14 or node.js 12 with the --experimental-modules flag enabled.

[pradeeptakdas]

Hi @michielvandergeest
Could you review this PR?

[dersanli]

Hi @michielvandergeest
Could you review this PR?

+1 pls

[wouterlucas]

Validated with Lightning app, still works with the NodeJS related changes. Will push this through.

[wouterlucas]

Please rebase so we can merge.

[wouterlucas]

LGTM