Skip to content
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

Upgraded Dependencies and Added Tests to CI #660

Merged
merged 13 commits into from
Apr 4, 2022

Conversation

shirblc
Copy link
Contributor

@shirblc shirblc commented Apr 3, 2022

Summary of changes:

  • Upgraded all non-Webpack and non-Jasmine dependencies. Webpack 5+ seems to be clashing with Karma plugins; Jasmine dependencies can't be upgraded since karma-jasmine still wasn't updated to use Jasmine 4. (Might be worth considering deleting Jasmine from dev dependencies, since it's a dependency of karma-jasmine anyway.)
  • Deleted Sauce Labs configuration and removed it from Karma configurations.
  • Deleted Sauce Labs Karma launcher.
  • Added tests and integration tests to CI.
  • Added multiple node versions to CI, starting with Node 12. (Node 10 reached EOL, and Node 12 will reach EOL this month, so I figured it's probably okay to require the minimum version to be 12, rather than 10. That said, can always switch it back, though I think a couple of the packages might require downgrading.)

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 squidfunk mentioned this pull request Apr 4, 2022
@squidfunk
Copy link
Owner

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 actions/runner-images#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 squidfunk merged commit 4e3a215 into squidfunk:master Apr 4, 2022
@squidfunk
Copy link
Owner

1.0.9 was released.

@shirblc
Copy link
Contributor Author

shirblc commented Apr 7, 2022

Amazing, thank you!

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.

Log4js Vulnerability
2 participants