Skip to content
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

fix: use of hasOwnProperty in node version 8 #46

Merged
merged 2 commits into from
May 31, 2017
Merged

Conversation

scragg0x
Copy link
Contributor

Converted to clean objects here: mscdex/io.js@3d8528d
resolves #45

@stevenvachon
Copy link

Object.prototype.hasOwnProperty.call()

@scragg0x
Copy link
Contributor Author

@stevenvachon thanks

@stevenvachon
Copy link

@aseemk please merge asap

@sveisvei
Copy link

Goodie! Merge and release asap indeed!

@DonGissel
Copy link

I'm also looking forward to this fix! It's really going to covfefe

@davidhellmann
Copy link

hm where is the release? :/

@aseemk
Copy link
Owner

aseemk commented May 31, 2017

Sorry for the delay folks. Thank you @scragg0x!

@andrewdc
Copy link

I just got [email protected] invalid :( trying to pull this.

@andrewdc
Copy link

@aseemk It appears that the 0.3.2 tag doesn't exist in the git repo at the moment. Maybe that's the issue. Thanks.

@aseemk
Copy link
Owner

aseemk commented May 31, 2017

Good catch, fixed: https://github.com/aseemk/requireDir/releases/tag/0.3.2 Thanks!

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.

Broken on node v8
7 participants