Skip to content

Incompatible with coffeescript 1.9.0 #175

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Incompatible with coffeescript 1.9.0 #175

wants to merge 1 commit into from

Conversation

@vilasmaciel
Copy link

@vilasmaciel vilasmaciel commented Mar 11, 2015

With newer versións of coffescript ( > 1.8.0 ) the code

class Listener
    constructor: (@options, emit) ->
        @app = new App()
        @app.options = options

compiles to

Listener = (function() {
    function Listener(options1, emit) {
      this.options = options1;
      this.handler = __bind(this.handler, this);
      this.app = new App();
      this.app.options = options;

returning a ReferenceError: options is not defined on the line 135 this.app.options = options;

@brycekahle brycekahle changed the title Force coffescript version to 1.8.0 Incompatible with coffeescript 1.9.0 Mar 11, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@vilasmaciel