lostluck commented on pull request #11979: URL: https://github.com/apache/beam/pull/11979#issuecomment-655910522
That works fine. Given how mechanical the change is, it wouldn't be unreasonable to do the change all at once in one PR. However, I do recommend doing all the directory renames in separate commits from the deleting the old mods so that we can check the relatively smaller changes on a per commit basis (rather than just all at once). Won't matter to the final merge, since it will get squashed, but it will make it easier to 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. For queries about this service, please contact Infrastructure at: [email protected]
