benbria / coffee-coverage

Istanbul and JSCoverage-style instrumentation for CoffeeScript files.
MIT License
145 stars 31 forks source link

fix expressions array in compile #96

Open orengriffin opened 6 years ago

orengriffin commented 6 years ago

fixes issue #93 this is a very naive fix.

coveralls commented 6 years ago

Coverage Status

Coverage increased (+3.6%) to 78.252% when pulling 27c2accc2918be7c4f238eca4daea2f5225cdbe4 on Optibus:fix-first-line into e80b5baed437e822561ac67ab40bf1611a4ae9fe on benbria:master.

coveralls commented 6 years ago

Coverage Status

Coverage increased (+3.6%) to 78.252% when pulling 27c2accc2918be7c4f238eca4daea2f5225cdbe4 on Optibus:fix-first-line into e80b5baed437e822561ac67ab40bf1611a4ae9fe on benbria:master.

coveralls commented 6 years ago

Coverage Status

Coverage increased (+3.6%) to 78.252% when pulling 27c2accc2918be7c4f238eca4daea2f5225cdbe4 on Optibus:fix-first-line into e80b5baed437e822561ac67ab40bf1611a4ae9fe on benbria:master.

elinskytap commented 6 years ago

Is it any updates to this PR? I got the same error in my project. Thanks in advance.

orengriffin commented 6 years ago

@elinskytap does this fix work for you?

jwalton commented 6 years ago

It's a bit of a hacky fix, but I'm stuck on coffeescript 1.x because when I try to upgrade to 2 my entire application doesn't even start anymore and I haven't invested the time to figure out why, so I'll merge it if it fixes things. :P

jwalton commented 6 years ago

(It would be nice if there was a test case to go along with this, though. ;)