chore: Remove serde feature

This feature was added with good intentions of reducing compilation time for users that don't need it, but the project in the current state doesn't actually compile without this feature. So we remove it to reduce the complexity of the codebase.

Author: Maximkaaa

galileo/Cargo.toml

@@ -12,7 +12,7 @@ readme = "../README.md"
 exclude = ["examples"]
 
 [features]
-default = ["wgpu", "serde", "winit", "_tests", "rustybuzz", "image", "reqwest-default-tls"]
+default = ["wgpu", "winit", "_tests", "rustybuzz", "image", "reqwest-default-tls"]
 wgpu = ["dep:wgpu", "raw-window-handle"]
 geojson = ["dep:geojson", "galileo-types/geojson"]
 rustybuzz = ["dep:rustybuzz"]
@@ -46,7 +46,7 @@ quick_cache = { workspace = true }
 raw-window-handle = { workspace = true, optional = true }
 regex = { workspace = true }
 rustybuzz = { workspace = true, optional = true }
-serde = { workspace = true, optional = true, features = ["std", "derive", "rc"] }
+serde = { workspace = true, features = ["std", "derive", "rc"] }
 serde_json = { workspace = true }
 strfmt = { workspace = true }
 thiserror = { workspace = true }

galileo/src/color.rs

@@ -1,10 +1,8 @@
-#[cfg(feature = "serde")]
 use serde::{Deserialize, Serialize};
 
 /// Color representation.
-#[derive(Debug, Default, Copy, Clone, PartialEq, Eq)]
-#[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
-#[cfg_attr(feature = "serde", serde(from = "String", into = "String"))]
+#[derive(Debug, Default, Copy, Clone, PartialEq, Eq, Serialize, Deserialize)]
+#[serde(from = "String", into = "String")]
 pub struct Color {
     r: u8,
     g: u8,