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

Change the cache size limit #27

Closed
wants to merge 1 commit into from

Conversation

ssuchanowski
Copy link

fixes #8

@special-character
Copy link

special-character commented Jun 5, 2020

@ssuchanowski This is in the built dist file. I think the source code is somewhere deeper in the cache lib. I am going to try and find out if an update to the cache package might fix this.

edit: Cache is currently cypress-io/github-actions-cache in package.json

@special-character
Copy link

I didn't solve this. Ended up being that this code is still in the latest used cache lib here: https://github.com/cypress-io/github-actions-cache/blob/master/src/save.ts#L57

@agilgur5
Copy link
Contributor

agilgur5 commented Aug 5, 2020

Potentially relevant, my PR #37 , changes to use the upstream @actions/cache as that is now an NPM package as well (previously it was only an action which is why the fork and branch existed). Sounds like that might possibly resolve this as well?

EDIT: yes, I believe it does as this check has increased to the current 5GB limit in the dist file of my PR (too big to view on GitHub directly, search fileSizeLimit in the raw)

@github-actions
Copy link

🎉 This issue has been resolved in version 1.4.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@ssuchanowski ssuchanowski deleted the patch-1 branch August 11, 2020 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

400mb limit is no longer needed
3 participants