[riot-notifications] [RIOT-OS/RIOT] sys/ubjson: deprecate module. (#11741)

Martine Lenders notifications at github.com
Tue Jun 25 15:48:28 CEST 2019


miri64 requested changes on this pull request.



> @@ -20,12 +20,16 @@
  * @defgroup sys_ubjson  Universal Binary JSON library
  * @ingroup  sys_serialization
  * @brief    Provides a library to read and write UBJSON serialized data
+ *
+ * @deprecated  This module is deprecated and will be removed in future releases.

I think this note should be enough. 

> @@ -20,12 +20,16 @@
  * @defgroup sys_ubjson  Universal Binary JSON library
  * @ingroup  sys_serialization
  * @brief    Provides a library to read and write UBJSON serialized data
+ *
+ * @deprecated  This module is deprecated and will be removed in future releases.

Also it should be specified which release at the earliest (usually 2 releases in the future, since a release is imminent I would say 2020.01 though) and given that we have two (`tinycbor` and `cn-cbor`) we should mention alternatives to port to.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/11741#pullrequestreview-254015765
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190625/8cadad88/attachment-0001.html>


More information about the notifications mailing list