adoroszlai commented on PR #4424: URL: https://github.com/apache/ozone/pull/4424#issuecomment-1480716251
When posting patches to Jira, old ones are usually kept. Force-pushing is like keeping only the latest patch. Keeping around old branches helps, but you have to let others know about them, and even then it's still not as streamlined as incremental commits. You are right that comments are still visible after force-push. However, it is hard to see the state of code they were referring to and how code was changed in response to them. https://www.freecodecamp.org/news/optimize-pull-requests-for-reviewer-happiness#request-a-review -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
