alamb commented on code in PR #6852:
URL: https://github.com/apache/arrow-rs/pull/6852#discussion_r1889102504


##########
README.md:
##########
@@ -82,6 +82,25 @@ versions approximately every 2 months.
 
 [`object_store`]: https://crates.io/crates/object_store
 
+### Deprecation Guidelines
+
+Minor releases may deprecate, but not remove APIs. Deprecating APIs allows
+downstream Rust programs to still compile, but generate compiler warnings. This
+gives downstream crates time to migrate prior to API removal.
+
+All deprecated APIs are marked using the `#[deprecated]` attribute with both 
the
+first version they were deprecated in, and what new API to use instead.
+
+```rust
+#[deprecated(since = "51.0.0", note = "Use `date_part` instead")]

Review Comment:
   I tried to do it in 9fcc60c365



##########
README.md:
##########
@@ -82,6 +82,25 @@ versions approximately every 2 months.
 
 [`object_store`]: https://crates.io/crates/object_store
 
+### Deprecation Guidelines
+
+Minor releases may deprecate, but not remove APIs. Deprecating APIs allows
+downstream Rust programs to still compile, but generate compiler warnings. This
+gives downstream crates time to migrate prior to API removal.
+
+All deprecated APIs are marked using the `#[deprecated]` attribute with both 
the

Review Comment:
   Clarified in 9fcc60c365



-- 
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: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to