sentry-javascript icon indicating copy to clipboard operation
sentry-javascript copied to clipboard
verified publisher icon getsentry

New Deno release 8.13.0 broke imports / has incorrect import details in README

Open Alexa-Green opened this issue 1 year ago • 3 comments

Environment

SaaS (https://sentry.io/)

Steps to Reproduce

  1. Try to import deploy using the instructions in the Deno README.
  2. Find that [sentry](https://deno.land/x/sentry/index.mjs) is no longer found.

Expected Result

The import for Sentry does not need to be changed after update to 8.13.0

Actual Result

https://deno.land/x/sentry/index.mjs is not found, and it should now be https://deno.land/x/sentry/build/index.mjs. Either documentation should be updated or contents should not be in a containing build folder. There is no documentation for this update so it broke deploys.

Product Area

Releases

Link

No response

DSN

No response

Version

No response

Alexa-Green avatar Jun 27 '24 18:06 Alexa-Green

Assigning to @getsentry/support for routing ⏲️

getsantry[bot] avatar Jun 27 '24 18:06 getsantry[bot]

Routing to @getsentry/product-owners-issues for triage ⏲️

getsantry[bot] avatar Jun 27 '24 23:06 getsantry[bot]

Hey @Alexa-Green apologies for the trouble.

This was a breaking change we accidentally released - we'll get it fixed for the next release of the SDK. Apologies for the trouble, you can use 8.12.0 for now to get around it.

AbhiPrasad avatar Jun 28 '24 16:06 AbhiPrasad

8.14.0 was released today, but this doesn't appear to be fixed. /index.mjs is still missing.

brc-dd avatar Jul 04 '24 15:07 brc-dd

Yes apologies, my fix wasn't robust enough, opened https://github.com/getsentry/sentry-javascript/pull/12773 to fix

AbhiPrasad avatar Jul 04 '24 16:07 AbhiPrasad

This is fixed in 8.15.0. Issue can be closed I guess.

brc-dd avatar Jul 08 '24 06:07 brc-dd