-
Notifications
You must be signed in to change notification settings - Fork 892
Fix: ensure repr_mimebundle handles empty include/exclude parameters gracefully
#7787
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
All contributors have signed the CLA ✍️ ✅ |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
I have read the CLA Document and I hereby sign the CLA |
repr_mimebundle handles empty include/exclude parameters gracefully|
@alonsosilvaallende would it be possible to add a small test for this so we don't regress it? |
akshayka
left a comment
There was a problem hiding this 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
|
Sure, I have added the required tests @mscolnick |
|
🚀 Development release published. You may be able to view the changes at https://marimo.app?v=0.19.3-dev19 |
📝 Summary
Closes #7786
I have followed the recommendations by @xflr6
🔍 Description of Changes
Before:

After:

📋 Checklist