4 ms·
I don't think that's a good argument. An inexperienced developer can inadvertently do a lot of damage in infinitely many ways, and your example is just one of t
by WCSTombs 2y ago
I don't think that's a good argument. An inexperienced developer can inadvertently do a lot of damage in infinitely many ways, and your example is just one of them. I don't think it's a good principle to structure a code base to protect inexperienced coders from themselves, which is always going to be a losing battle.
This may have been a missed opportunity for a teachable moment:
- The new hire most likely used `git commit -a` without checking `git status` or `git diff`, which is bad practice.
- The new hire probably didn't even know about their global .gitignore.
IMO a better way to steer new developers in the right direction is simply code review, and I'd think even a pretty cursory review would have caught those .DS_Store files.
- aniforprez 2y agoWe can do both those things and also take away their ability to add unwanted files which is exactly what we did. Why even allow things like this to happen when a single line to a gitignore solves it forever? The reason we saw those .DS_Store files in the first place was during PR review