[PR #39] [MERGED] Replace serde_yaml with serde_norway due to maintenance concerns #40

New issue

Closed

opened 2026-05-22 22:05:17 +01:00 by JakeStanger · 0 comments

JakeStanger commented

2026-05-22 22:05:17 +01:00

Owner

📋 Pull Request Information

Original PR: https://github.com/corn-config/corn/pull/39
Author: @puppymati
Created: 3/5/2025
Status: ✅ Merged
Merged: 3/5/2025
Merged by: @JakeStanger

Base: master ← Head: serde-yaml

📝 Commits (1)

7134030 Switch from serde_yaml to serde_norway

📊 Changes

6 files changed (+21 additions, -21 deletions)

View changed files

📝 Cargo.lock (+15 -15)
📝 corn-cli/Cargo.toml (+1 -1)
📝 corn-cli/src/bin/corn.rs (+1 -1)
📝 corn-cli/src/error.rs (+2 -2)
📝 libcorn/Cargo.toml (+1 -1)
📝 libcorn/tests/parser_tests.rs (+1 -1)

📄 Description

serde_yaml has been deprecated and is no longer maintained. After evaluating alternatives, serde_norway appears to be the most actively maintained fork at present. While not perfect, it provides a more sustainable dependency than an unmaintained library.

The current YAML ecosystem in rust is rather bleak, with several alternatives having various drawbacks. For context on the situation:

In the longer term I am considering rolling my own lib, but this change should address the immediate concern.

Depends on #38

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/corn-config/corn/pull/39 **Author:** [@puppymati](https://github.com/puppymati) **Created:** 3/5/2025 **Status:** ✅ Merged **Merged:** 3/5/2025 **Merged by:** [@JakeStanger](https://github.com/JakeStanger) **Base:** `master` ← **Head:** `serde-yaml` --- ### 📝 Commits (1) - [`7134030`](https://github.com/corn-config/corn/commit/713403063a28e67189f9395b3bfb34111e164b0e) Switch from serde_yaml to serde_norway ### 📊 Changes **6 files changed** (+21 additions, -21 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.lock` (+15 -15) 📝 `corn-cli/Cargo.toml` (+1 -1) 📝 `corn-cli/src/bin/corn.rs` (+1 -1) 📝 `corn-cli/src/error.rs` (+2 -2) 📝 `libcorn/Cargo.toml` (+1 -1) 📝 `libcorn/tests/parser_tests.rs` (+1 -1) </details> ### 📄 Description serde_yaml has been deprecated and is no longer maintained. After evaluating alternatives, serde_norway appears to be the most actively maintained fork at present. While not perfect, it provides a more sustainable dependency than an unmaintained library. The current YAML ecosystem in rust is rather bleak, with several alternatives having various drawbacks. For context on the situation: - https://www.reddit.com/r/rust/comments/1ibdxf9/beware_of_this_guy_making_slop_crates_with_ai/ - https://nitter.poast.org/davidtolnay/status/1883906113428676938 - https://github.com/rustsec/advisory-db/issues/2132 In the longer term I am considering rolling my own lib, but this change should address the immediate concern. Depends on #38 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

JakeStanger added the

pull-request

label

2026-05-22 22:05:17 +01:00

JakeStanger referenced this issue

2026-05-22 22:05:17 +01:00

[PR #40] [MERGED] Restructure codebase to follow modern Rust practices #41

JakeStanger closed this issue

2026-05-22 22:05:17 +01:00