search

turbos-finance / turbos-clmm-sdk

6 stars 5 forks source link

ESM things #2

Closed losman0s closed 1 year ago

losman0s commented 1 year ago

I'm not an expert on JS bundling & co, but is "type": "module" in package.json absolutely required? I was playing around with the SDK today and had to change it to "type": "commonjs" to get anything to work in a simple Node script.

runcat1234567 commented 1 year ago

Actually, we support both ESM and commonJS bundle files.

but is "type": "module" in package.json absolutely required

Yes, it's a mistake, I will remove it from package.json later.