rocketraman commented on PR #65:
URL: 
https://github.com/apache/logging-log4j-kotlin/pull/65#issuecomment-1937349716

   > @rocketraman, I would appreciate it if you don't force-push. This renders 
all historical conversations detached, since associated commits don't exist 
anymore.
   
   @vy  Yeah, GitHub is quite poor at managing code reviews with force pushes, 
unfortunately. My preference is generally to sacrifice the GitHub PR 
cleanliness in favor of a clean commit history, as the former is transient and 
the latter is forever. However, I recognize this is personal preference and 
most people tend to prefer keeping the PR clean and not caring too much about 
the commit log. Its unfortunate that GitHub isn't better at this, given that 
even git.git upstream prefers the force push / clean commit history approach.


-- 
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...@logging.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to