search

mrhenry / polyfill-library

MIT License
7 stars 1 forks source link

update browsers list #81

Closed romainmenke closed 1 week ago

romainmenke commented 2 weeks ago

I was not seeing any failures for IE 11 in other projects using Browserstack. A difference between this and those projects is that I favor more recent OS's in others. While this config uses the oldest OS.

The failures we are seeing appear to always be IE 11 on Windows 7. While other projects run just fine with IE 11 on Windows 10.

This change sorts the list of browser configs so that more recent OS's appear first and are then picked first when running tests.

I hope that this resolves our test failures.

mhassan1 commented 2 weeks ago

It looks like ios/10 (iPhone 7) is failing consistently. I see that BrowserStack will be removing it in two weeks, so maybe we can comment it out in this PR.

romainmenke commented 2 weeks ago

Thank you for spotting that! Removing it early makes sense