Skip to content

Conversation

@dmadisetti
Copy link
Collaborator

📝 Summary

Previous comments were removed on jupyter conversion, this will maintain them

@vercel
Copy link

vercel bot commented Sep 24, 2025

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

Project Deployment Preview Comments Updated (UTC)
marimo-docs Ready Ready Preview Comment Sep 24, 2025 10:30pm

@app.cell
def _(np):
# Cell 7: Duplicate import with different comment
# This should be deduplicated but preserve one of the comments # Import numpy again with different comment
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Line removal leaves things a bit strange

mscolnick
mscolnick previously approved these changes Sep 24, 2025
@dmadisetti dmadisetti added the bug Something isn't working label Sep 24, 2025
@dmadisetti dmadisetti linked an issue Sep 24, 2025 that may be closed by this pull request
@dmadisetti dmadisetti merged commit 9ed2b19 into main Sep 24, 2025
40 of 41 checks passed
@dmadisetti dmadisetti deleted the dm/comment-preserve branch September 24, 2025 23:11
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.

marimo convert discards comments on certain transformations

3 participants