jaredpalmer / tsdx

Zero-config CLI for TypeScript package development
https://tsdx.io
MIT License
11.28k stars 507 forks source link

Sourcemaps for extractErrors plugin #184

Open swyxio opened 5 years ago

swyxio commented 5 years ago

Current Behavior

currently our transformErrorMessages plugin doesnt provide a source map. this breaks sourcemap behavior downstream.

Desired Behavior

sourcemapz

Suggested Solution

add teh sourcemap

Who does this impact? Who is this for?

all extractErrors users

Additional context

clues we can use to implement this

jaredpalmer commented 5 years ago

I’m not going to have time to dig into this for a while. Would love for someone to jump in and help.

swyxio commented 5 years ago

ya i dont consider this very high priority at the moment and may revisit when this bothers me enough. its just interesting that the fb rollup/babel implementation doesnt do any sourcemap specific code and it Just Works

edit: ah. they turn off sourcemaps??