PR template clutters commit messages

microsoft / vcpkg

C++ Library Manager for Windows, Linux, and MacOS

MIT License

23.41k stars 6.47k forks source link

PR template clutters commit messages #37352

Open dg0yt opened 8 months ago

dg0yt commented 8 months ago

The current style of merging PRs makes all parts of the GH PR template enter the git log, including parts hidden in GH UI (). In general, commit messages are not given much attention (except from those who dig history for cruft blocking good port quality).

Osyotr commented 8 months ago

Last commit without clutter: 6978381401d33a5ad6a3385895d12e383083712a First commit with clutter: a56d892ac792797d779eae79d3423e0131422628

dg0yt commented 6 months ago

@BillyONeal This doesn't seem to get sufficient attention.

It is annoying when searching git history.
It affects forks of the repo.

github-actions[bot] commented 4 days ago

This is an automated message. Per our repo policy, stale issues get closed if there has been no activity in the past 180 days. The issue will be automatically closed in 14 days. If you wish to keep this issue open, please add a new comment.

dg0yt commented 4 days ago

I acknowledge that the commit message are very short since June. I assume this is implemented as manual step in the merge process. If you think this issue isn't needed to track improving the operation, feel free to close it.