-
Notifications
You must be signed in to change notification settings - Fork 76
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
idea: run expensive automated tests daily rather than on each PR #1408
Comments
How long do our CI tests take? To require this I feel like they'd need to be long enough to significantly delay merges, or the team would have to be big enough for the merge queue to fall behind. |
this issue in particular is about figuring out a threshold duration for tests (i.e. a max length one would reasonably want to wait for tests before being able to hit merge) and then making tests that take longer happen once daily rather than on every merge. it was inspired by @robknight's answer to one of my question on his #1403 PR, where I asked if it would be possible to run the bundling/npm-publishing test script in CI, to which his answer was that it would probably take too long. I am not proposing we move all our tests into a daily CI. |
I don't really know what "too long" is, but I do know that we had an issue for making CI faster, which suggests that we don't want it to go significantly slower than it does now. One other option might be to have different CI jobs for |
I have discovered a way of making this much faster, fwiw. |
No description provided.
The text was updated successfully, but these errors were encountered: