Skip to content

Conversation

@alonsosilvaallende
Copy link
Contributor

📝 Summary

Closes #7786
I have followed the recommendations by @xflr6

🔍 Description of Changes

Before:
marimo-graphviz-before

After:
marimo-graphviz-after

📋 Checklist

  • [ X] I have read the contributor guidelines.
  • [ X] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on Discord, or the community discussions (Please provide a link if applicable).
  • [ X] Tests have been added for the changes made.
  • [ X] Documentation has been updated where applicable, including docstrings for API changes.
  • [ X] Pull request title is a good summary of the changes - it will be used in the release notes.
@github-actions
Copy link

github-actions bot commented Jan 12, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@vercel
Copy link

vercel bot commented Jan 12, 2026

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

Project Deployment Review Updated (UTC)
marimo-docs Ready Ready Preview, Comment Jan 13, 2026 9:42am
@alonsosilvaallende
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@alonsosilvaallende alonsosilvaallende changed the title Ensure repr_mimebundle handles empty include/exclude parameters gracefully Jan 12, 2026
mscolnick
mscolnick previously approved these changes Jan 12, 2026
@mscolnick mscolnick added the bug Something isn't working label Jan 12, 2026
@mscolnick
Copy link
Contributor

@alonsosilvaallende would it be possible to add a small test for this so we don't regress it?

Copy link
Contributor

@akshayka akshayka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks so much for the fix, @alonsosilvaallende. I have left one comment

@alonsosilvaallende
Copy link
Contributor Author

Sure, I have added the required tests @mscolnick

@mscolnick mscolnick merged commit 17bc1ee into marimo-team:main Jan 13, 2026
26 of 40 checks passed
@github-actions
Copy link

🚀 Development release published. You may be able to view the changes at https://marimo.app?v=0.19.3-dev19

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

3 participants