Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: alternative array includes #857

Merged
merged 1 commit into from
Apr 25, 2020
Merged

fix: alternative array includes #857

merged 1 commit into from
Apr 25, 2020

Conversation

kazupon
Copy link
Owner

@kazupon kazupon commented Apr 25, 2020

closes #851

@codecov-io
Copy link

Codecov Report

Merging #857 into v8.x will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             v8.x     #857   +/-   ##
=======================================
  Coverage   96.40%   96.41%           
=======================================
  Files          10       10           
  Lines         863      864    +1     
=======================================
+ Hits          832      833    +1     
  Misses         31       31           
Impacted Files Coverage Δ
src/components/number.js 100.00% <100.00%> (ø)
src/index.js 97.88% <100.00%> (ø)
src/util.js 98.33% <100.00%> (+0.02%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6e70740...e564190. Read the comment docs.

@kazupon kazupon merged commit dea111b into v8.x Apr 25, 2020
@kazupon kazupon deleted the fix/includes branch April 25, 2020 18:02
@kazupon kazupon added Type: Bug Bug or Bug fixes and removed PR: merged labels Apr 25, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Type: Bug Bug or Bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Array.includes() breaks in IE 11
2 participants