Skip to content

[fix] - Polyfill missed Array.includes in test helper. #233

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

Merged
merged 1 commit into from
May 7, 2017

Conversation

beefancohen
Copy link
Contributor

Forgot one use in a test helper, not sure how CI passed. It failed when I published v5.0.1. It shouldn't affect the fix in production, but running npm test inside the dependency directory will fail for certain node versions.

Issue first brought up in #230

@coveralls
Copy link

coveralls commented May 7, 2017

Coverage Status

Coverage remained the same at 99.547% when pulling 83063c4 on array-includes into 47b1306 on master.

@beefancohen beefancohen merged commit c16fa00 into master May 7, 2017
@ljharb ljharb deleted the array-includes branch May 7, 2017 19:02
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.

2 participants