I recently finished reading The Pragmatic Programmer by Andy Hunt and Dave Thomas and despite the vast amount of good advice, one phrase stuck out to me: "no broken windows".
My understanding is that there was a study done which showed that a building with one unrepaired window soon had multiple unrepaired windows, and other signs of neglect. The idea being that once people saw that no one cared enough to fix the first window, there was less of a desire to maintain the rest of the building, because, again, no one cared about the window.
Hunt and Thomas extend this idea this idea to software engineer, suggesting that as software engineers, we should not let neglect slip into our codebases.
I've often found myself seeing something and thinking "someone else will get to this" or "this is an issue, but not large enough to warrant addressing". Lately, since I've read this book, I've stopped thinking like that. I remind myself, "no broken windows". Fix the issue now, or at least address the issue in an upcoming PR.
It's been my experience that unclean codebases can tend to run rampant when not enough care is put into maintaining them. That care needs to be put in constantly, and truthfully, it requires active decision making and thinking that not everyone wants to engage in. I think in the long run though, the whole team will be glad if they can look at a well maintained and easy to understand codebase.
Fix any broken windows.