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

ci(*): more efficient testing #2068

Open
wants to merge 4 commits into
base: next
Choose a base branch
from
Open

ci(*): more efficient testing #2068

wants to merge 4 commits into from

Conversation

cabljac
Copy link
Contributor

@cabljac cabljac commented Apr 19, 2024

Currently our CI builds ALL the extensions and runs ALL the tests, even on pull requests.

This proposed workflow filters on changes in the PR, and only runs tests on extensions which have changes.

There are still some small optimisations i'd like to make eventually, but:

  1. We can still run all the tests on pushes to next. In PRs to next we can just test the PR's changes.
  2. Currently test logs are super noisy, this slows us down as it's actually hard in the github action logs to work out which tests are failing.
  3. This also runs the tests in parallel, so it'd (potentiall) be quicker even if you made changes in every extension.

@cabljac cabljac force-pushed the @invertase/smart-ci branch 23 times, most recently from f84cbc9 to 560dcd4 Compare April 19, 2024 15:01
@cabljac cabljac requested a review from a team as a code owner April 19, 2024 15:01
@cabljac cabljac force-pushed the @invertase/smart-ci branch 6 times, most recently from 0d39f8c to 0180a1c Compare April 19, 2024 15:28
@cabljac cabljac force-pushed the @invertase/smart-ci branch 20 times, most recently from 0c01eb6 to ff2c7a2 Compare April 23, 2024 16:58
@cabljac cabljac force-pushed the next branch 3 times, most recently from c8885bd to d1481a2 Compare April 24, 2024 10:27
@cabljac cabljac removed the do not merge Do not merge this Pull Request label May 7, 2024
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.

None yet

1 participant