Skip to content
This repository was archived by the owner on Dec 8, 2024. It is now read-only.

Support instrumenting coffeescript #166

Open
jdfree opened this issue Mar 18, 2014 · 5 comments
Open

Support instrumenting coffeescript #166

jdfree opened this issue Mar 18, 2014 · 5 comments

Comments

@jdfree
Copy link

jdfree commented Mar 18, 2014

The ibrik project (https://github.com/Constellation/ibrik) has added some coffeescript support to istanbul, but is incomplete. It does, however, have a nice instrumenter.coffee. With minimal effort, this instrumenter could be compiled to js and used by istanbul so that istanbul's instrument command can instrument either js or coffee using the same command that presently only handles js. Operations that make use of "instrument" + "report" (for example, this: https://github.com/airportyh/testem/tree/master/examples/coverage_istanbul) would then have full coffeescript support.

@willhoag
Copy link

willhoag commented Apr 9, 2014

+1

@jigfox
Copy link

jigfox commented Mar 9, 2015

+1
Any news on this except for #167 which isn't merge yet

@lo1tuma
Copy link

lo1tuma commented Mar 10, 2015

👎
I think istanbul should not support coffescript directly, there are a lot of other compile-to-js languages (e.g. typescript, jsx). The best thing would be if istanbul would support generic preprocessors.

@Zolmeister
Copy link

+1

@lo1tuma see #167 for why source-maps aren't sufficient

@silkentrance
Copy link

@gotwarlost transpiling first + sourcemaps will do the trick. Please close as this is basically just a usage issue. Thanks!

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 a pull request may close this issue.

6 participants