search

godot-rust / gdnative

Rust bindings for Godot 3
https://godot-rust.github.io
MIT License
3.61k stars 210 forks source link

`serde_yaml` end of life #1073

Closed Bromeon closed 5 months ago

Bromeon commented 5 months ago

serde_yaml is no longer maintained (last release, Reddit).

It's likely a matter of time until cargo-deny will start complaining about it.

We only use it in a small test for serde:

https://github.com/godot-rust/gdnative/blob/7e55836e53bfd1ccd618939ec30ab6bf2400b614/test/src/test_serde.rs#L167-L179

It's probably OK if we remove that part and the dependency. Opinions?

chitoyuu commented 5 months ago

That should be fine, I agree. What we should have used was probably serde_test, but the current enumeration of formats is still more than enough, even with YAML removed from it.