nickva commented on PR #4355: URL: https://github.com/apache/couchdb/pull/4355#issuecomment-1373978465
Good idea to encourage using rebases if possible, especially for small or medium PR. But it could just be an admonishment rather than an enforcement. I can see in cases when large features are merged a merge commit might make sense. Another thing I often watch out for is that I avoid merging main into my PR branch and then merging back into main. That often ends up looking rather confusing in git history. -- 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: notifications-unsubscr...@couchdb.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org