2 ms·
That’s a pretty reductive attitude towards linters. If writing Go (for example), linters will tell you if you have ineffective assignments, or have neglected to
by jen20 5y ago
That’s a pretty reductive attitude towards linters. If writing Go (for example), linters will tell you if you have ineffective assignments, or have neglected to check an error value - both things that may well block releases.
While one could argue these are the job of the compiler, they are not today, and so linting during CI is not only appropriate, but practically required.
- fivea 5y ago> While one could argue these are the job of the compiler, they are not today, and so linting during CI is not only appropriate, but practically required. I disagree. Stylistic choices should not block a pipeline. It's impossible to argue in favour of blocking a release just because there's a whitespace in a place where it makes some random developer frown. The robustness principle also applies here. And no, build/critical errors are not the same as stylistic/linting errors.
- jen20 5y agoWhether you check errors is a correctness issue, not a stylistic one.
- giovannibonetti 5y agoThe fact that Go requires a linter to ensure errors are properly handled is a whole other topic
- jen20 5y agoI don’t disagree there.