inline `applicationinsights` as ESM module

huang-julien / nitro-applicationinsights

Application insights plugin for nitro.

MIT License

10 stars 0 forks source link

inline `applicationinsights` as ESM module #59

Open pi0 opened 5 months ago

pi0 commented 5 months ago

(related: #58)

Hi. Following up issues reported re CJS/ESM syntax detection, and the fact that applicationinsights even for v3 is using legacy syntax, i would suggest to either move it to upstream issue or we can simply use unbuild setup to inline applicationinsights as a pure ESM module.

Please let me know if you like me to try a PR for this :)

huang-julien commented 5 months ago

Hi :wave: there's already an upstream issue in app insights which didn't get any answer from the team.

I'd love to get a PR from you 🤩 !

This is probably the last fix before moving to nitro modules :grin: