Thanks for feedback an your are exactly right. This applies for new module/project and every point you mentioned is valid.
We should also make sure these rules applies to bug fixes, or code adding as feature in existing files. I raised all these question my self and taken initial step. As a first step, developed wrapper tool (POC) to apply flake8 on patches only which we can immediately adopt to Basetools etc.. Along with the POC tool(if possible we can make as flake8 plugin) , I will come up with presentation to address all our concerns. We can review and finalize the best practices. Thanks, Kondal. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#55655): https://edk2.groups.io/g/devel/message/55655 Mute This Topic: https://groups.io/mt/71786757/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
