@vzarytovskii makes sense and done.

There is only one downside I see with raw Stream extensions versus having an abstraction that would remove the concern of 'use/IDisposable' being handled properly.

This may be improved by having extensions to IFileSystem that would deal with it, taking a Stream -> 't function instead of returning the raw Stream.

Since this is going to be used in all tooling, turning the API this way (for some use cases at least) would reduce concern for tooling on top of FCS to misuse the resources the concrete implementation returns.

Green.

Bump

Hey @Happypig375, not sure this is a priority for being merged.

I like the change because it skips recompiles but it is not a big deal and probably not urgent for the maintainers to adjust it (also, conficting files now, so can't be merged as is).

I had the pending PR with init properties which is more important that I intended to update based off this, but I can also unblock it by manually adjusting the surface area .fs file there.

@smoothdeveloper Sorry it took so long. I've fixed the build + merged the main. If it looks ok to you, going to merge once green.

@vzarytovskii thanks! it looks green & good to me.