Skip to content

Conversation

@dmadisetti
Copy link
Collaborator

📝 Summary

The following fails in script mode:

with app.setup:
    if False:
        value = ...

Since we naively expect value to be assigned. This PR makes an exception for setup cells in script mode, enabling this.

@vercel
Copy link

vercel bot commented Dec 31, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
marimo-docs Ready Ready Preview, Comment Dec 31, 2025 4:27pm

@dmadisetti dmadisetti changed the title Dm/fix partial setup fix: invalid error thrown when undefined setup variables Dec 31, 2025
mscolnick
mscolnick previously approved these changes Dec 31, 2025
@dmadisetti dmadisetti merged commit b70293a into main Dec 31, 2025
37 of 41 checks passed
@dmadisetti dmadisetti deleted the dm/fix-partial-setup branch December 31, 2025 17:09
@dmadisetti dmadisetti added the bug Something isn't working label Jan 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants