Skip to content

Conversation

dineshyv
Copy link
Contributor

@dineshyv dineshyv commented Nov 14, 2024

Screenshot 2024-11-13 at 10 10 08 PM

Copy link
Contributor

@raghotham raghotham left a comment

Choose a reason for hiding this comment

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

Great!

@dineshyv dineshyv merged commit 5f95581 into main Nov 14, 2024
3 checks passed
@dineshyv dineshyv deleted the dineshyv/use-rich branch November 14, 2024 06:21
@ashwinb
Copy link
Contributor

ashwinb commented Nov 14, 2024

woohoo 💌

@ashwinb
Copy link
Contributor

ashwinb commented Nov 14, 2024

Oh - can we not print empty Metadata as {} it looks a bit funny. None or just leaving it empty would be better.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants