Skip to content

fix: add a has_warnings parse enum to indicate formatting errors#8288

Merged
dmadisetti merged 1 commit intomainfrom
dm/soft-warning
Feb 12, 2026
Merged

fix: add a has_warnings parse enum to indicate formatting errors#8288
dmadisetti merged 1 commit intomainfrom
dm/soft-warning

Conversation

@dmadisetti
Copy link
Collaborator

📝 Summary

Previously, all parsing "Violations" would provide a warning indicating the file would change on save. As many of these "Violations" are just formatting issues, we change the warning in these cases.

Previously:

W: This notebook has errors, saving may lose data. Continuing anyway.

Now:

I: This notebook has minor issues that will be auto-corrected on save.

@vercel
Copy link

vercel bot commented Feb 12, 2026

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

Project Deployment Actions Updated (UTC)
marimo-docs Ready Ready Preview, Comment Feb 12, 2026 11:17pm

Request Review

@dmadisetti dmadisetti requested a review from mscolnick February 12, 2026 23:17
@dmadisetti dmadisetti added the bug Something isn't working label Feb 12, 2026
@dmadisetti dmadisetti merged commit 6eedabf into main Feb 12, 2026
47 of 50 checks passed
@dmadisetti dmadisetti deleted the dm/soft-warning branch February 12, 2026 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

2 participants