GitHub user mengw15 added a comment to the discussion: Merge the Python Notebook Migration Tool in a Single PR
@chenlica Based on the discussion, here's my proposed conclusion: **We don't go with a single PR. Instead, we split into multiple PRs, with a feature flag to hide the feature until everything is merged.** The thread converged on a few points that together resolve the original concern: 1. Smaller, incremental PRs are preferred over one big single PR (per @Yicong-Huang's development loop). 2. Partial functionality at the merge level is fine, as long as users don't see incomplete state. 3. we can utilize a feature flag to achieve this(default false until all PRs are merged) GitHub link: https://github.com/apache/texera/discussions/4590#discussioncomment-16777139 ---- This is an automatically sent email for [email protected]. To unsubscribe, please send an email to: [email protected]
