Skip to content
This repository has been archived by the owner on May 1, 2020. It is now read-only.

fix(sourcemaps) copy vendor.js.map #1396

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix(sourcemaps) copy vendor.js.map #1396

wants to merge 1 commit into from

Conversation

HugoHeneault
Copy link

@HugoHeneault HugoHeneault commented Feb 22, 2018

Short description of what this resolves:

Keep vendor.js.map instead of ignoring it

Changes proposed in this pull request:

  • For unkown (and uncommented) reason, vendor.js.map was skipped on sourcemaps copy
  • I removed this filtering

Fixes: # might partially fix #1235 (for the missing vendor part)

@imhoffd
Copy link
Contributor

imhoffd commented Sep 27, 2018

@HugoHeneault I think we excluded it by default in the beginning because the generated source map file is usually pretty large.

It may be better to have this behind an option?

@HugoHeneault
Copy link
Author

@dwieeb Sure! If I remember right I think polyfill.js was also missing!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sourcemaps appear to be broken when uglify/--prod task is used.
2 participants