Skip to content

fix expressions array in compile#96

Open
orengriffin wants to merge 1 commit into
benbria:masterfrom
Optibus:fix-first-line
Open

fix expressions array in compile#96
orengriffin wants to merge 1 commit into
benbria:masterfrom
Optibus:fix-first-line

Conversation

@orengriffin
Copy link
Copy Markdown

fixes issue #93
this is a very naive fix.

@coveralls
Copy link
Copy Markdown

Coverage Status

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

2 similar comments
@coveralls
Copy link
Copy Markdown

Coverage Status

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

@coveralls
Copy link
Copy Markdown

Coverage Status

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

@elinskytap
Copy link
Copy Markdown

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

@orengriffin
Copy link
Copy Markdown
Author

@elinskytap does this fix work for you?

@jwalton
Copy link
Copy Markdown
Contributor

jwalton commented Oct 24, 2018

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
Copy link
Copy Markdown
Contributor

jwalton commented Oct 24, 2018

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants