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

Do not use turbo cache on main #2098

Merged
merged 3 commits into from
Jul 25, 2022
Merged

Do not use turbo cache on main #2098

merged 3 commits into from
Jul 25, 2022

Conversation

mattseddon
Copy link
Member

@mattseddon mattseddon commented Jul 25, 2022

We've been filling up a cache on main that we (generally) never hit. It is now causing jobs to timeout. I've added a conditional statement to not use the cache on main.

@mattseddon mattseddon marked this pull request as ready for review July 25, 2022 21:58
@codeclimate
Copy link

codeclimate bot commented Jul 25, 2022

Code Climate has analyzed commit b9ccc29 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (85% is the threshold).

This pull request will bring the total coverage in the repository to 96.7%.

View more on Code Climate.

@mattseddon mattseddon merged commit 5b6446b into main Jul 25, 2022
@mattseddon mattseddon deleted the no-cache-on-main branch July 25, 2022 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants