[riot-notifications] [RIOT-OS/RIOT] makefiles: add code generator targets (#8573)

Alexandre Abadie notifications at github.com
Tue May 12 12:51:43 CEST 2020


> I like this tool but I would like it documented somewhere in out doc, maybe in getting started?

Good idea, I'll push a commit asap.

> I haven't looked at the python code, but this doesn't seem relevant here, the functionality is what we want here.

I improved it slightly during the last days: now it uses Jinja2 templating engine to render the files (I should have done that from the start...). There's still room for improvement/refactoring and I'm now in the process of adding tests.

> we are not reviewing external code here. If the project is merged into the organization then we can review it IMO.

If the community is OK to maintain such a tool in a separate repository, that would be great indeed.

-- 
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/8573#issuecomment-627266565
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20200512/3c463ba5/attachment.htm>


More information about the notifications mailing list