hailin0 commented on PR #4369: URL: https://github.com/apache/incubator-seatunnel/pull/4369#issuecomment-1476254910
> > > Hi. Checking the commit message doesn’t make any sense to me because we are using “Squash and merge”, all the commit messages will be rewrote to the pull request title when merging, checking the unused commit messages increase the contributors’ burden while bring nothing. > > > > > > You are correct. However, currently, some committers may not modify the final message when merging code, which means that the information merged into the main branch is the last submission message of the contributor. If standardizing committers' merging behavior is important and needs to be addressed, do you have any suggestions? > > The default commit message when merging is the pull request title, so you might want to check PR title instead +1 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
