squidfunk / karma-viewport

A Karma plugin for testing responsive features and layout
MIT License
49 stars 14 forks source link

Upgraded Dependencies and Added Tests to CI #660

Closed shirblc closed 2 years ago

shirblc commented 2 years ago

Summary of changes:

Haven't enabled codecov as I don't have the credentials, but otherwise I've tested it both locally and in GitHub Actions, and all appeared to be working correctly. Only other thing is, I don't think there's full support for all the browsers previously listed in the Sauce Labs configuration... we might have change the environment to run tests against pre-Chromium Edge (based on this)

Closes #659

squidfunk commented 2 years ago

Thanks for this PR! Awesome work! I agree with all points you mentioned and say that it's good to merge.

Only other thing is, I don't think there's full support for all the browsers previously listed in the Sauce Labs configuration... we might have change the environment to run tests against pre-Chromium Edge (based on https://github.com/actions/virtual-environments/issues/97)

As mentioned in #659 I think it's okay to switch to the headless strategy, which should yield much more stable builds. SauceLabs led to very flaky tests. If a we find that karma-viewport breaks in a browser it should support, we'll just fix it on the way. Being a dev dependency, it's not critical anyway.

Would you like to be a collaborator of this project? My time to spend here is very limited, and given the great work you've done here, I'm happy to add you as collaborator, so you can fix issues as they appear. Just ping me then and I'll issue a new release (which I will also do in a minute)

squidfunk commented 2 years ago

1.0.9 was released.

shirblc commented 2 years ago

Amazing, thank you!