GitHub user sshiv012 added a comment to the discussion: [Notebook Migration WIP] Adding R support
I think trusting the selected tab when notebook metadata is missing is ok. The tab is already an explicit user declaration of intent, so I’d treat metadata as an extra signal, not the primary source of truth. I’d prefer what you suggested over failing on missing metadata because notebook metadata is not always reliable or complete, and rejecting otherwise valid notebooks would add friction without really improving correctness. The only thing I’d consider is making the error wording clear that it’s a mismatch rather than saying the notebook itself is invalid, like maybe “This notebook appears to use Python, but you selected R notebook import.” GitHub link: https://github.com/apache/texera/discussions/8866#discussioncomment-18806066 ---- This is an automatically sent email for [email protected]. To unsubscribe, please send an email to: [email protected]
