irelephant [he/him]🍭 to Programmer [email protected]English • 8 months agolooks good to melemm.eeimagemessage-square39fedilinkarrow-up1696arrow-down18cross-posted to: [email protected]
arrow-up1688arrow-down1imagelooks good to melemm.eeirelephant [he/him]🍭 to Programmer [email protected]English • 8 months agomessage-square39fedilinkcross-posted to: [email protected]
minus-square@[email protected]linkfedilink23•8 months agoMeanwhile in another universe one of my biggest win was to introduce this line in our PR validation pipeline. eslint . --max-warnings 0
minus-square@[email protected]linkfedilink3•8 months agoIn a codebase with a lot of warnings is even better for me to add a disable comments for all the existing warning and then not allow any new one in. And then each time a part of the code needs to be touched the existing warning there should be solved too.
minus-square@[email protected]linkfedilink2•8 months agoSeveral times I’ve set the max warnings to whatever the current warning count is, and then decreased that over time.
Meanwhile in another universe one of my biggest win was to introduce this line in our PR validation pipeline.
eslint . --max-warnings 0deleted by creator
In a codebase with a lot of warnings is even better for me to add a disable comments for all the existing warning and then not allow any new one in.
And then each time a part of the code needs to be touched the existing warning there should be solved too.
Several times I’ve set the max warnings to whatever the current warning count is, and then decreased that over time.