@KentonVarda big pet peeve I have is the “changes made” section they tend to add, which is a lossy and redundant version of the diff. reviewers should review…
@housecor meh. I'd rather catch errors as early as possible. If I need to commit some work in progress I got `gwip` which aliases --no-verify. gets obnoxious if it takes too long but quick sanity checks are very welcome.
While it is true that complexity must be kept minimal, "code is a liability" is the wrong framing. Blindly minimizing LOCs can easily lead to more complexity, not less. Common mistake.
Code is a liability, not an asset. So goal of software engineer is delivering the maximum amount of desired functionality at the cost of the least amount of code complexity, even as desired functionality evolves over time.