Skip to content

feat: cacheMode option - #267

Merged
erickzhao merged 11 commits into
electron:mainfrom
dsanders11:feat/dont-cache
Jul 15, 2024
Merged

feat: cacheMode option#267
erickzhao merged 11 commits into
electron:mainfrom
dsanders11:feat/dont-cache

Conversation

@dsanders11

@dsanders11 dsanders11 commented Dec 14, 2023

Copy link
Copy Markdown
Member

Closes #209. This is less of a problem these days as @electron/fiddle-core actually steals the file out of the cache, so there's not duplicate copies, but that's not optimal behavior, and leaves behind SHASUMS256.txt files in the cache unnecessarily. So, provide an option to not add the downloaded file to the cache.

Closes electron/packager#1552

@dsanders11
dsanders11 marked this pull request as ready for review January 18, 2024 18:10
@dsanders11
dsanders11 requested a review from a team as a code owner January 18, 2024 18:10
@VerteDinde
VerteDinde self-requested a review February 7, 2024 18:52
Comment thread src/types.ts Outdated
Comment thread src/index.ts Outdated
Comment thread src/index.ts Outdated
@MarshallOfSound

Copy link
Copy Markdown
Member

@dsanders11 @erickzhao got bored at the airport, updated this to use an enum and such

@erickzhao
erickzhao self-requested a review July 12, 2024 20:00
@dsanders11 dsanders11 changed the title feat: dontCache option Jul 13, 2024
@dsanders11

Copy link
Copy Markdown
Member Author

I can't approve my own PR, but LGTM. 👍

@MarshallOfSound MarshallOfSound left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can approve it lol

@erickzhao erickzhao left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

impl looks great

Comment thread src/types.ts
Comment thread src/types.ts Outdated
MarshallOfSound and others added 3 commits July 13, 2024 11:37
Co-authored-by: Erick Zhao <erick@hotmail.ca>
Co-authored-by: Erick Zhao <erick@hotmail.ca>
@erickzhao
erickzhao merged commit 0adc5b9 into electron:main Jul 15, 2024
@continuous-auth

Copy link
Copy Markdown

🎉 This PR is included in version 3.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4 participants