@PackElend, I've noticed that several applicants commit changes unrelated to their PR. It's been a time sink in some PR as we have to remind them over and over again to remove these changes.
So I think we should suggest they check their changes carefully before committing them. Something like:
Before committing your changes, make sure you check that what you are going to commit. For example, some editors reformat the code when saving, or you might have added test code that shouldn't end up in the PR.
Use git diff or git difftool to see exactly what's changed and remove any change that's not relevant to your issue. Consider using git add -p to add only the relevant changes.
Pull requests that include changes unrelated to your issue will not be merged. Moreover it does not reflect well on your work when mentors have to remind you to clean up your pull request.
2 Likes
