Absolutely correct.

I always tell our new developers: If your new feature works, you aren't done, you have to check if your code has to be refined first, before checking it in. (e.g. duplicated passages made into a common function, ugly hacks removed and "done properly", stuff like this) Documentation and testing are also mandatory, but that's because of the industry we work in.