Add zstd support
~~Opening as draft as I haven't tested it yet or haven't written any tests yet, but I'd be too interested in zstd support.~~
Closes #1463
~~EDIT: :disappointed: rustfmt stopped working again. I'll fix it when I'll submit tests~~
:roll_eyes: Windows https://github.com/facebook/zstd/pull/3061#issuecomment-1034196613
It seems like something has been broken with DNS for the entire day :thinking:
Should I disable testing zstd on Windows so that we can get this merged? It doesn't look like there's been any progress on the zstd side, and I don't have a Windows machine to debug the issue
Is zstd supposed to work on Windows? Is it a non-goal? Or they haven't been able to spend resources fixing it?
It is supposed to work on Windows but I have no idea why it's not :sweat_smile:. Maybe https://github.com/Nemo157/async-compression/pull/144 will fix it.
I got it to work now @seanmonstar
indexmap bumped MSRV to 1.56
Will the sync client also be getting this support?
Will the sync client also be getting this support?
It will. Now that I look at it I might have forgotten to add the option to disable it for it :sweat_smile:
Awesome, thanks!