Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

emit with sourceMaps #69

Closed
cdaringe opened this issue Aug 21, 2022 · 1 comment · Fixed by #113
Closed

emit with sourceMaps #69

cdaringe opened this issue Aug 21, 2022 · 1 comment · Fixed by #113

Comments

@cdaringe
Copy link

Problem

emit used to support sourceMaps. The new API is drastically more constrained than the prior, preventing me from generating mappable src.

Discussion

Can the compiler options be re-exposed back into emit?

@Ciantic
Copy link

Ciantic commented Sep 23, 2022

On latest it's on by default, I'm trying to turn it off.

yacinehmito added a commit to yacinehmito/deno_emit that referenced this issue Apr 4, 2023
Since the initial release, the bundle type, the compiler options and the
imports were not properly forwarded from JS to Rust.

We now forward them, and added some tests to verify this behaviour.

Closes denoland#83, denoland#69, denoland#29.
yacinehmito added a commit to yacinehmito/deno_emit that referenced this issue Apr 9, 2023
Since the initial release, the bundle type, the compiler options and the
imports were not properly forwarded from JS to Rust.

We now forward them, and added some tests to verify this behaviour.

Closes denoland#83, denoland#69, denoland#29.
yacinehmito added a commit to yacinehmito/deno_emit that referenced this issue Apr 25, 2023
Since the initial release, the bundle type, the compiler options and the
imports were not properly forwarded from JS to Rust.

We now forward them, and added some tests to verify this behaviour.

Closes denoland#83, denoland#69, denoland#29.
yacinehmito added a commit to yacinehmito/deno_emit that referenced this issue Apr 26, 2023
Since the initial release, the bundle type, the compiler options and the
imports were not properly forwarded from JS to Rust.

We now forward them, and added some tests to verify this behaviour.

Closes denoland#83, denoland#69, denoland#29.
dsherret pushed a commit that referenced this issue Apr 26, 2023
Forward arguments from JS to Rust

Since the initial release, the bundle type, the compiler options and the
imports were not properly forwarded from JS to Rust.

We now forward them, and added some tests to verify this behaviour.

Closes #83, #69, #29.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants