[riot-notifications] [RIOT-OS/RIOT] SAUL: Reduce RAM requirements (#11392)
notifications at github.com
Mon Apr 15 12:02:57 CEST 2019
I agree with the usage (and changes) of `fmt` the other changes I find a bit controversial. Mainly that adding a new sensor now requires to change 2 `enums` instead of one, however I like the general idea of having a class and start numbering from there.
TL;DR: please split this in 2 PRs one for `fmt` (which I would merge directly) and one for the other change which (to me) need some discussion, for instance @leandrolanzieri might have an opinion on this too.
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications