GitHub user zuozhiw added a comment to the discussion: Merge the Python Notebook Migration Tool in a Single PR
Agree with Yicong's point, best practices is merge PRs gradually that are easy to review, I recognize the overhead to the author, but small PRs are the best way. If a PR breaks the main branch primary functionality, like running a workflow, then it's better to merge things together. But this PR doesn't seems to be the case and this feature is rather isolated so it's ok to have it in partial state. GitHub link: https://github.com/apache/texera/discussions/4590#discussioncomment-16777211 ---- This is an automatically sent email for [email protected]. To unsubscribe, please send an email to: [email protected]
