Quashing commits might be good practice, depending on the circumstances. Quashing commits is a way to organize your commit history before allowing other programmers to access the source code. While quashing commits makes the codebase concise it can still cause your team to lose some valuable information that existed in the unintegrated branches. It comes down to personal preferences but think carefully before you decide to do it.