[riot-devel] [riot-maintainer] Code Quality Task Force

Oleg Hahm oliver.hahm at inria.fr
Mon Mar 6 18:15:04 CET 2017


Hi again,

On Mon, Mar 06, 2017 at 03:02:29PM +0100, Kaspar Schleiser wrote:
> 1. move all vendor headers into a clearly recognizable sub directory
> containing "/vendor/", for easy exclusion (Hauke will do that)

Tackled in #6700.

> 
> 2. Activate CI failing on all doxygen warnings. We'll do that in #6688.
> (Oleg will do that).

Enabled. 

> 3. Pull request (and merge) fixes to #6688 until all warnings are gone
> and the PR passes CI, then merge the PR.
 #6688 is rebased on #6700 now and includes the patches from Raphael and
 Thomas. Thanks! Down to 11 warnings.

> 4. create a PR similar to #6688, but for cppcheck warnings. Same fixing
> procedure.

I will do this later tonight.
 
> 6. meet up on next RIOT Summit and have a beer together.

The best part of the TF! ;-)

Cheers,
Oleg
-- 
panic("esp_handle: current_SC == penguin within interrupt!");
        linux-2.2.16/drivers/scsi/esp.c
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.riot-os.org/pipermail/devel/attachments/20170306/4e8967d2/attachment.sig>


More information about the devel mailing list