Not writing (and updating) documentation explaining intent and purpose.
I work in a small team (2 front-end devs), and my co-worker is a bit picky about documenting every function, class, method, etc etc.
But sometimes all he documents is something a function name tells it upfront, I really think there's no purpose in doing something like:
/** This function transforms lower case string to uppper case **/
If you declare functions and variables with intuitive names, sometimes there's no need to explain them :)
It is called self-documenting functions for a reason right?
yup... unless that function transforms the string using some alien computation, there's no need for that
We're a place where coders share, stay up-to-date and grow their careers.
We strive for transparency and don't collect excess data.