broccoli-coffee icon indicating copy to clipboard operation
broccoli-coffee copied to clipboard

added support for sourcemap

Open ming-codes opened this issue 9 years ago • 14 comments

It turns out CoffeeScript support embedded source map.

So I just added a switch in options to additionally embed the CoffeeScript source into the output. This should resolve #1

ming-codes avatar Oct 17 '14 05:10 ming-codes

Hey, this sounds cool. Any reasons against the merge (apart from the unresolved conflict)?

kraftwer1 avatar Jan 26 '15 21:01 kraftwer1

Yes, can we get this merged? What needs to be done to move it forward?

nathanpalmer avatar Jan 27 '15 14:01 nathanpalmer

FYI, there's some relevant discussion happening in joliss/broccoli-uglify-js#8.

iangreenleaf avatar Jan 28 '15 05:01 iangreenleaf

Fixed conflict :smiley:

ming-codes avatar Jan 28 '15 06:01 ming-codes

(Also tracking https://github.com/jashkenas/coffeescript/issues/3678 in case the CoffeeScript compiler gains native support for data URLs at some point.)

joliss avatar Jan 29 '15 23:01 joliss

Great! Thanks, I'll try it out.

kraftwer1 avatar Jan 30 '15 06:01 kraftwer1

+1

GeoffreyBooth avatar Mar 02 '15 19:03 GeoffreyBooth

Hi! What is the status of this?

ShogunPanda avatar Apr 21 '15 00:04 ShogunPanda

+1

Rrrapture avatar Apr 29 '15 22:04 Rrrapture

+5

chainlink avatar May 14 '15 16:05 chainlink

@joliss Unless you think this is going to break anything, I think it's better to merge this. I think inaction in this case is actually doing more harm than good. If this implementation of source map is not good enough, we can always have other contributors to fix it.

ming-codes avatar May 28 '15 01:05 ming-codes

It seems @joliss was waiting for the comments left in commits to be fixed, but then abandoned the project for those last months? That's quite sad, since the issue is apparently fixed 5 months ago...

igorsantos07 avatar May 31 '15 20:05 igorsantos07

+1

varblob avatar Jul 11 '15 10:07 varblob

Any chance of releasing this?

marnen avatar Apr 21 '16 17:04 marnen