Unbreak changelogs (#4332)

Author: bugadani

esp-hal/CHANGELOG.md

@@ -26,7 +26,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - ESP32: Expose `psram_vaddr_mode` via `PsramConfig` (#3990)
 - ESP32-S3: Expose more `Camera` config options (#3996)
 - Functions to query chip revision for every chip (#3892)
-- ESP32-S3: Add RtcI2c driver (#0000)
+- ESP32-S3: Add RtcI2c driver (#4016)
 - `ShaBackend, Sha<N>Context`: Work-queue based SHA driver (#4013)
 - I2S: `i2s::master::Config` with support for more TDM mode standards (#3985)
 - ESP32: support outputting the main I2S clock signal (#4128)

esp-radio/CHANGELOG.md

@@ -32,10 +32,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - `esp-ieee802154` package has been folded into `esp-radio`, it's now alloc. (#3861, #3890)
 - `ble`, `esp-now`, `csi`, `sniffer`, `esp-ieee802154` and `smoltcp` features and APIs marked as unstable (#3865)
 - Update bt-hci version to add additional HCI commands (#3920)
-- A number of enums/structs have been marked as `#[non_exhaustive]` (#3981, #4017)
-  - `AuthMethod`, `Protocol`, `AccessPointInfo`, `AccessPointConfiguration`, `ClientConfiguration`, `Capability`, `Configuration`, `WifiEvent`, `InternalWifiError`, `ScanTypeConfig`, `WifiState`, and `WifiMode`
+- `AuthMethod`, `Protocol`, `AccessPointInfo`, `AccessPointConfiguration`, `ClientConfiguration`, `Capability`, `Configuration`, `WifiEvent`, `InternalWifiError`, `ScanTypeConfig`, `WifiState`, and `WifiMode` have been marked as `#[non_exhaustive]` (#3981, #4017)
 - The `Configuration`, `ClientConfiguration`, `AccessPointConfiguration`, and `EapClientConfiguration` enums have been renamed to `ModeConfig`, `ClientConfig`, `AccessPointConfig`, and `EapClientConfig` (#3994, #4278)
-  - Error types implements `core::error:Error`
+- Error types implements `core::error:Error` (#3994, #4278)
 - Use `esp-phy` internally for PHY initialization (#3892)
 - `ap_state()` and `sta_state()` marked as stable (#4017)
 - `wifi_state()` marked as unstable (#4017)

xtask/src/changelog.rs

@@ -188,9 +188,11 @@ impl ChangelogForVersion {
                 current_group.lines.push(line);
             } else if line.trim().is_empty() {
                 // Empty line, just skip
-            } else {
-                // Not ours any more.
+            } else if line.starts_with("[") {
+                // Link definitions in footer
                 break;
+            } else {
+                bail!("Changelog line {} starts with unexpected character", line);
             }
             lines.next();
         }
@@ -273,9 +275,20 @@ impl ChangelogLine {
                 .trim_end_matches('.') // normalize "(#pr)."
                 .trim_end_matches(')')
                 .split(',')
-                .map(|pr| pr.trim().trim_start_matches('#').parse::<usize>())
-                .collect::<Result<Vec<_>, _>>()
-                .with_context(|| format!("Could not parse PR number {prs}"))?
+                .map(|pr| {
+                    pr.trim()
+                        .trim_start_matches('#')
+                        .parse::<usize>()
+                        .with_context(|| format!("Could not parse PR number {prs}"))
+                })
+                .map(|pr| {
+                    if let Ok(0) = pr {
+                        bail!("PR number cannot be zero")
+                    } else {
+                        pr
+                    }
+                })
+                .collect::<Result<Vec<_>, _>>()?
         } else if indentation == 0 {
             bail!("Missing PR number");
         } else {

xtensa-lx/CHANGELOG.md

@@ -23,7 +23,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Changed
 
-* This crate no longer uses the `links` field in its `Cargo.toml` (#3697)
+- This crate no longer uses the `links` field in its `Cargo.toml` (#3697)
 
 ## [v0.11.0] - 2025-06-03