From: Matt Corallo <git@bluematt.me>
Date: Thu, 20 May 2021 20:09:29 +0000 (+0000)
Subject: Add note in CONTRIBUTING.md that some structs need prefix/suffixes
X-Git-Tag: v0.0.98~18^2
X-Git-Url: http://git.bitcoin.ninja/index.cgi?a=commitdiff_plain;h=bdb2d220e85e29779475a7ba172835404dea3e98;hp=02c57f59404d3f4ae33c7bca0a39c58444d1ad0d;p=rust-lightning

Add note in CONTRIBUTING.md that some structs need prefix/suffixes
---

diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
index 9ddd792b..3cf463ba 100644
--- a/CONTRIBUTING.md
+++ b/CONTRIBUTING.md
@@ -113,6 +113,14 @@ rustup component add clippy
 cargo clippy
 ```
 
+Significant structures that users persist should always have their serialization methods (usually
+`Writeable::write` and `ReadableArgs::read`) begin with
+`write_ver_prefix!()`/`read_ver_prefix!()` calls, and end with calls to
+`write_tlv_fields!()`/`read_tlv_fields!()`.
+
+Updates to the serialized format which has implications for backwards or forwards compatibility
+must be included in release notes.
+
 Security
 --------