Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
[rdf4j-dev] REMINDER: Please use meaningful commit messages and/or squash before you PR

I am having a hell of time cherrypicking things from the master branch to prep a release. On some feature branches there’s tons of commits with useless messages like “Fixes” or “code review” or “Merged”. 

Can I remind everyone to always use, at the very least, the issue number in the message (preferably right at the front), so it’s easy for me to spot, in the history, which commits to pick for a certain fix?

If you can’t be bothered doing this all the time in your day-to-day workflow, that’s fine, but in that case please make sure that you squash your commits on your local branch/fork before you submit it for a PR - that way, you can clean up your commit messages “after the fact” (and it also make it easier to cherrypick, btw :)).  

I may in future request you do a squash on your PR as part of the review.

Thanks,

Jeen

Back to the top