tvalentyn commented on PR #38362:
URL: https://github.com/apache/beam/pull/38362#issuecomment-4410620647

   process nits:
   - there is a lot of commits on this pr, before submitting a PR for review 
you can squash commits, unless you think it would be helpful to review 
individual commits
   - between review iterations there is no need to rebase, unless there are 
merge conflicts; you can add new commits to the branch (without squashing new, 
unreviewed commits into reviewed commits); then github makes it easier to 
review commits that were added since the last review iteration


-- 
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]

Reply via email to