-
Notifications
You must be signed in to change notification settings - Fork 58
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.
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
instance instanceof mixin? #17
Comments
I cannot confirm your bug. I just tested it with
on Windows 7 x64 The test code file being: 'use strict';
const mx = require('mixwith');
let MyMixin = mx.Mixin((superclass) => class extends superclass {
constructor() {
super(...arguments);
}
});
class MyClass extends mx.mix(Array).with(MyMixin) {
constructor() {
super();
}
}
var a = new MyClass();
console.log(a instanceof Array);
console.log(a instanceof MyClass);
console.log(a instanceof MyMixin); with the console output:
|
I'm using node v6.9.1 (npm v3.10.8) and got the same error: |
I get same error |
It's because you need to wrap your mixins with |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is there a way to do something like:
Where the result would be:
Currently it returns:
The text was updated successfully, but these errors were encountered: