Idag började det dyka upp lite frågor om kodlinting och kodstil om de råd man får från phpmd och phpstan. Det är trevligt att se och det är verkligen bra att intressera sig för vad lintern säger och klura ut om man håller med.
Källan till att förstå lintern är det regelverket de använder och de kommentarer som finns kring varje regel.
När man senare eventuellt kommer fram till att man vill tillåta en viss varning så kan man supressa det varningsmeddelandet.
Det är fritt fram att suppressa/ignorera varningar. När man gör det får man en form av dokumentation i sin kod som visar att man tillåter en kodsekvens som kan ifrågasättas, men man har tittat lite extra på den och väljer att tillåta den.
Så brukar jag själv göra. Jag vill gärna att utskriften från linters skall vara tom och inga fel visas. Då känns det bra.
En annan variant är att jobba med en "baseline" med ett antal fel man väljer att acceptera, man sparar dessa i en baseline fil och därefter visar lintern bara nya fel. Både phpmd och phpstan stödjer arbete med en baseline.
Idag började det dyka upp lite frågor om kodlinting och kodstil om de råd man får från phpmd och phpstan. Det är trevligt att se och det är verkligen bra att intressera sig för vad lintern säger och klura ut om man håller med.
Källan till att förstå lintern är det regelverket de använder och de kommentarer som finns kring varje regel.
När man senare eventuellt kommer fram till att man vill tillåta en viss varning så kan man supressa det varningsmeddelandet.
Det är fritt fram att suppressa/ignorera varningar. När man gör det får man en form av dokumentation i sin kod som visar att man tillåter en kodsekvens som kan ifrågasättas, men man har tittat lite extra på den och väljer att tillåta den.
Så brukar jag själv göra. Jag vill gärna att utskriften från linters skall vara tom och inga fel visas. Då känns det bra.
En annan variant är att jobba med en "baseline" med ett antal fel man väljer att acceptera, man sparar dessa i en baseline fil och därefter visar lintern bara nya fel. Både phpmd och phpstan stödjer arbete med en baseline.