rancher / charts-build-scripts

Apache License 2.0
9 stars 32 forks source link

Switch to using a memfs instead of an osfs #92

Open aiyengar2 opened 1 year ago

aiyengar2 commented 1 year ago

Currently, charts-build-scripts uses a osfs:

https://github.com/rancher/charts-build-scripts/blob/2901446ef556324ce10bdf88889181516d45dd08/pkg/filesystem/filesystem.go#L24

However, ideally we should be making loading the current directory into a memfs, making all calls in memory, and then only modifying the underlying filesystem after executing all operations.

https://github.com/go-git/go-billy/blob/205ba6585ca03bd60e304f7fa0b270232ba1d707/memfs/memory.go#L28-L31