{"action":"created","issue":{"url":"https://api.github.com/repos/facebook/react/issues/3632","labels_url":"https://api.github.com/repos/facebook/react/issues/3632/labels{/name}","comments_url":"https://api.github.com/repos/facebook/react/issues/3632/comments","events_url":"https://api.github.com/repos/facebook/react/issues/3632/events","html_url":"https://github.com/facebook/react/pull/3632","id":67274040,"number":3632,"title":"Allow mixins to be es6 classes.","user":{"login":"jsfb","id":9595985,"avatar_url":"https://avatars.githubusercontent.com/u/9595985?v=3","gravatar_id":"","url":"https://api.github.com/users/jsfb","html_url":"https://github.com/jsfb","followers_url":"https://api.github.com/users/jsfb/followers","following_url":"https://api.github.com/users/jsfb/following{/other_user}","gists_url":"https://api.github.com/users/jsfb/gists{/gist_id}","starred_url":"https://api.github.com/users/jsfb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jsfb/subscriptions","organizations_url":"https://api.github.com/users/jsfb/orgs","repos_url":"https://api.github.com/users/jsfb/repos","events_url":"https://api.github.com/users/jsfb/events{/privacy}","received_events_url":"https://api.github.com/users/jsfb/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/facebook/react/labels/CLA%20Signed","name":"CLA Signed","color":"e7e7e7"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2015-04-09T03:41:43Z","updated_at":"2015-04-20T23:41:58Z","closed_at":"2015-04-20T23:41:58Z","pull_request":{"url":"https://api.github.com/repos/facebook/react/pulls/3632","html_url":"https://github.com/facebook/react/pull/3632","diff_url":"https://github.com/facebook/react/pull/3632.diff","patch_url":"https://github.com/facebook/react/pull/3632.patch"},"body":"Allow mixins to be es6 classes.\r\n\r\nSide note: I had to downgrade two lint rules to warnings because apparently eslint (in it's current form) doesn't understand ES6 classes.  Meh, sucks.  We should get our eslint install fixed or file a ticket on eslint.  We never ran into this before because we only used es6 classes in tests and have linting disabled completely for tests?"},"comment":{"url":"https://api.github.com/repos/facebook/react/issues/comments/94593279","html_url":"https://github.com/facebook/react/pull/3632#issuecomment-94593279","issue_url":"https://api.github.com/repos/facebook/react/issues/3632","id":94593279,"user":{"login":"zpao","id":8445,"avatar_url":"https://avatars.githubusercontent.com/u/8445?v=3","gravatar_id":"","url":"https://api.github.com/users/zpao","html_url":"https://github.com/zpao","followers_url":"https://api.github.com/users/zpao/followers","following_url":"https://api.github.com/users/zpao/following{/other_user}","gists_url":"https://api.github.com/users/zpao/gists{/gist_id}","starred_url":"https://api.github.com/users/zpao/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zpao/subscriptions","organizations_url":"https://api.github.com/users/zpao/orgs","repos_url":"https://api.github.com/users/zpao/repos","events_url":"https://api.github.com/users/zpao/events{/privacy}","received_events_url":"https://api.github.com/users/zpao/received_events","type":"User","site_admin":false},"created_at":"2015-04-20T23:41:58Z","updated_at":"2015-04-20T23:41:58Z","body":"I don't think classes are the right conceptual model for mixins so enabling it feels like a step backward.\r\n\r\nLet's revisit classes / whatever as mixins when ES* has support or when we decide to make `React.createMixin` the way forward.\r\n\r\n>  Furthermore, prior to this change, all functions get thrown as ReactComponents, which is incorrect and confusing to someone who does try using an ES6 class here.\r\n\r\nWe can fix the error message to be more accurate."}}