Ämne på Diskussion:Developer guidelines

Automatiska komplexitetstest

7
Sebastian Berlin (WMSE) (diskussionbidrag)

Det kunde vara värt att kolla på om det finns något bra verktyg för att testa komplexitet automatiskt. Jag har ingen förhoppning om att det ska fånga allt perfekt, men det skulle kunna vara bra som ett steg innan manuell granskning. Stötte t.ex. på Code Climate som gör bl.a. detta och är "Free for open-source.".

André Costa (WMSE) (diskussionbidrag)

Jag har använt Code Climate för vissa projekt (hostade på Github). Det jag tyckte att den var väldigt bra på var att flagga nästan-duplicerad kod vilket ger en indikation på var refaktorering kan vara bra. Jag vill minnas att det var lite strul med att köra över default-inställningarna men annars är det inte ett dåligt verktyg att ha i lådan. Går även att integrera med webhooks i github så att man får feedback innan man mergar en PR.

André Costa (WMSE) (diskussionbidrag)
Sebastian Berlin (WMSE) (diskussionbidrag)

Kollade som hastigast och det ser ut som att en del saker där kan vara användbara. Främst tycker jag att varningar för duplicering (som du nämner ovan) och cognitive complexity ser intressanta ut. Hårda gränser för antalet rader i funktioner vet jag inte om de är lika användbara, men kan funka som varningsflaggor.

André Costa (WMSE) (diskussionbidrag)

Jo jag reagerade också på att långa filer/funktioner får så pass stränga straff.

Ska bli intressant hur exemplet ovan påverkas av refactor-js patchen.

Sebastian Berlin (WMSE) (diskussionbidrag)

Jag får inte "Trends" att fungera. Det är bara en laddningssnurra, oavsett vilken typ jag väljer.

André Costa (WMSE) (diskussionbidrag)