denoland / deno_emit

Transpile and bundle JavaScript and TypeScript under Deno and Deno Deploy
https://jsr.io/@deno/emit
MIT License
223 stars 23 forks source link

fix: properly set compiler options defaults #112

Closed yacinehmito closed 1 year ago

yacinehmito commented 1 year ago

In #108, the defaults of CompilerOptions were changes so we would stop emitting with inline source maps by default. This works when providing an empty object for compilerOptions, but not when not providing compilerOptions. This is because we used to fallback to deno_ast's defaults for EmitOptions when compilerOptions was not provided.

This PR fixes it; not providing compilerOptions is now equivalent to providing an empty object.

yacinehmito commented 1 year ago

@dsherret It's ready.