Skip to content

Conversation

@indexzero
Copy link
Contributor

There are a few motivations here:

  1. this is valid when binding to class functions in ES6.
  2. this inside of describe for mocha (and other test frameworks) is valid.

@rxmarbles
Copy link
Member

👍

indexzero added a commit that referenced this pull request Feb 8, 2017
…5. This aligns with future usage of `babel/no-invalid-this`. Fixes #39.
@indexzero
Copy link
Contributor Author

Merging this so that we can publish a minor before we formally deprecate after #40.

@indexzero indexzero merged commit 18f02a6 into master Feb 8, 2017
@indexzero indexzero deleted the this-class-is-valid branch February 8, 2017 03:38
indexzero added a commit that referenced this pull request Feb 8, 2017
…5. This aligns with future usage of `babel/no-invalid-this`. Fixes #39.
indexzero added a commit that referenced this pull request Feb 8, 2017
…5. This aligns with future usage of `babel/no-invalid-this`. Fixes #39.
indexzero added a commit that referenced this pull request Feb 8, 2017
…5. This aligns with future usage of `babel/no-invalid-this`. Fixes #39.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants