[riot-notifications] [RIOT-OS/RIOT] sys/metadata: Add a metadata command to let an application give metadata (#10961)

Kaspar Schleiser notifications at github.com
Wed Feb 6 14:50:00 CET 2019


I just realized that the module outputs in json format...

Look, my first reaction was as @miri64's: "what metadata?" 

I know how this works. You're happily solving some problem (in this case, maybe you've been working on the unified testing RDM) and need some functionality. It has something to do with metadata, and 15min later there's a PR for "sys/metadata".

This unfortunately blanks out that there's a plethora of different understandings for metadata. The PR description is not very helpful (it could mention json format, and that it is supposed to be used by the tests, ...). So as is, there's "metadata" in "sys/", but noone gets what the folder is there for.

I suspect there are gonna be more related functions. So how about directly creating a "sys/test" module? IMO, it is the right place for a function to print test metadata. It could be the in-tree code support for the coming RDM.
Soon there'll be "test_print_result_value()", "test_print_success()", ... Surely we don't want "sys/result_value" as module.

-- 
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/10961#issuecomment-461029157
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190206/bd522eef/attachment.html>


More information about the notifications mailing list