I wish there was a linter that stopped my colleagues from adding in a half-gigabyte of Apache Spark artifacts to go with a single line of code, which makes the product impossible to deploy to customers;
It sounds like this would be simple to catch during code reviews, or in the very least with a regression test running after the packaging stage.
or to implement another fucking O(n^2) algorithm that flies through the test suite and craps out in production;
It sounds like the job for a performance test.
or our placement students from trying to get ChatGPT shit through code review.
Aren’t code reviews catching this?
I also fucking hate checkstyle, or any of its friends like Google’s spotless; sometimes you want to format your code so that the underlying thinking is more obvious, perhaps to highlight how some parts are different and make things that are not as you would expect stand out, but no.
I’m afraid that’s not supposed to be handled at the code formatting level. That’s supposed to be handled by extracting code to a method or a dedicated class, and cover with unit tests to illustrate how it’s expected to work. If you’re trying to handle that with non-standard code formatting then I’m afraid you’re writing bad code.
It sounds like this would be simple to catch during code reviews, or in the very least with a regression test running after the packaging stage.
It sounds like the job for a performance test.
Aren’t code reviews catching this?
I’m afraid that’s not supposed to be handled at the code formatting level. That’s supposed to be handled by extracting code to a method or a dedicated class, and cover with unit tests to illustrate how it’s expected to work. If you’re trying to handle that with non-standard code formatting then I’m afraid you’re writing bad code.