search

aws / amazon-chime-sdk-js

A JavaScript client library for integrating multi-party communications powered by the Amazon Chime service.
Apache License 2.0
699 stars 472 forks source link

Remove extraneous tests from PR workflow #2862

Closed hensmi-amazon closed 2 months ago

hensmi-amazon commented 3 months ago

Issue #: None

Description of changes: We have fewer PRs nowadays so keeping a bunch of parallel sauce lab sessions available just for tests that are extremely unlikely to break doesn't make much sense. We still run these tests daily and on release, and they still can be run locally. We just need enough tests to sanity check that nothing explodes during a PR. I do not consider the content share tests to likely catch issues.

Testing:

N/A

Checklist:

  1. Have you successfully run npm run build:release locally? y

  2. Do you add, modify, or delete public API definitions? If yes, has that been reviewed and approved? n

  3. Do you change the wire protocol, e.g. the request method? If yes, has that been reviewed and approved? n

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

michhyun1 commented 3 months ago

Sounds good to me

hensmi-amazon commented 2 months ago

Closing for now since we renewed with enough extra sessions to handle PRs