tmchow opened a new pull request, #3051: URL: https://github.com/apache/iggy/pull/3051
## Summary Adds `#[serde(rename_all = "snake_case")]` to `SystemSnapshotType` and `SnapshotCompression` enums in `core/common/src/types/snapshot/mod.rs`. ## Changes These two enums were missing the serde rename attribute that all other enums in `core/common/src/types/` use. The `Display` and `FromStr` impls already produce snake_case strings (e.g., `filesystem_overview`, `process_list`), so serde serialization should match. Pattern used by other enums in the same directory: - `consumer_kind.rs:49` — `#[serde(rename_all = "snake_case")]` - `user_status.rs:26` — `#[serde(rename_all = "snake_case")]` - `identifier/mod.rs:53` — `#[serde(rename_all = "snake_case")]` - `message/partitioning_kind.rs:28` — `#[serde(rename_all = "snake_case")]` ## Testing `cargo fmt --all -- --check` passes. No JSON fixtures or config files reference these enum variants in PascalCase. ## AI Usage This contribution was developed with AI assistance (Claude Code + Codex). Fixes #2699 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
