fixed issue with newer IOS devices causing an error.

cssinjs / caniuse-support

Query the caniuse offline database for feature support.

MIT License

8 stars 6 forks source link

fixed issue with newer IOS devices causing an error. #55

Closed dipunm closed 4 years ago

dipunm commented 4 years ago

Steps to reproduce:

const { detectBrowser } = require('caniuse-support');
detectBrowser('Mozilla/5.0 (iPhone; CPU iPhone OS 12_2 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) Mobile/15E148');

results in an error: Cannot read property 'toString' of undefined.

coveralls commented 4 years ago

Pull Request Test Coverage Report for Build 547

1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 92.553%

Totals
Change from base Build 542:	0.2%
Covered Lines:	54
Relevant Lines:	58

💛 - Coveralls

AleshaOleg commented 4 years ago

@dipunm thanks for PR. Looks legit

AleshaOleg commented 4 years ago

Released in 1.0.3