grafana / pyroscope-nodejs

Pyroscope NodeJS integration
Apache License 2.0
27 stars 21 forks source link

Fix issue #32 with regenerator-runtime/runtime import #33

Closed bradleyheubel closed 3 hours ago

bradleyheubel commented 1 year ago

Fix issue NodeJS cannot find module regenerator-runtime/runtime #32

CLAassistant commented 1 year ago

CLA assistant check
All committers have signed the CLA.

aptomaKetil commented 5 months ago

Is this project abandoned? Pyroscope remains broken for Node projects using ESM.

Bruno-DaSilva commented 4 months ago

FYI after the big recent refactor, this now requires all relative paths (there are many more now) to end in .js.

nwalters512 commented 3 hours ago

Now that #89 is merged, this is obsolete.

bryanhuhta commented 3 hours ago

Closing because of https://github.com/grafana/pyroscope-nodejs/pull/89 getting merged.