Skip to content
New issue

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

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

Already on GitHub? # to your account

ncc run to use sourcemaps by default #230

Merged
merged 1 commit into from
Jan 18, 2019
Merged

ncc run to use sourcemaps by default #230

merged 1 commit into from
Jan 18, 2019

Conversation

guybedford
Copy link
Contributor

I think by default having the good debugging experience makes sense here, as the sourcemaps generation time isn't too long at all.

@codecov-io
Copy link

Codecov Report

Merging #230 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #230   +/-   ##
=====================================
  Coverage      70%    70%           
=====================================
  Files          13     13           
  Lines         670    670           
=====================================
  Hits          469    469           
  Misses        201    201
Impacted Files Coverage Δ
src/cli.js 0% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5b54d32...7026628. Read the comment docs.

@rauchg rauchg merged commit ecd00e2 into master Jan 18, 2019
@rauchg rauchg deleted the run-sourcemaps branch January 18, 2019 18:43
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants