Skip to content
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

Add trait implementations for Vec<T>, [T], Option<T> #421

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

nik-here
Copy link

No description provided.

@nik-here nik-here changed the title Add trait implementation for Vec<T>, [T], Option<T> Add trait implementations for Vec<T>, [T], Option<T> Feb 20, 2024
@nik-here
Copy link
Author

The CI on the main branch seems to fail generally. #1556. So this Pull request will probably be blocked until the CI errors on main branch are fixed.

@14ROVI
Copy link

14ROVI commented Jul 30, 2024

This looks good, and super useful. Hopefully it can be merged soon :D

@sorcix
Copy link

sorcix commented Feb 23, 2025

I actually like that Option<T> isn't supported out of the box. Usually when I encounter the error that an Option<T> can't be rendered, it is a mistake on my side. In those cases more than just the value has to be hidden when it is None. (e.g. enclosing markup)

The implementation in this MR would just hide the value.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants