Skip to content

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

Open
nik-here wants to merge 4 commits into
lambda-fairy:mainfrom
nik-here:main
Open

Add trait implementations for Vec<T>, [T], Option<T>#421
nik-here wants to merge 4 commits into
lambda-fairy:mainfrom
nik-here:main

Conversation

@nik-here

Copy link
Copy Markdown

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
Copy Markdown
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

14ROVI commented Jul 30, 2024

Copy link
Copy Markdown

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

@sorcix

sorcix commented Feb 23, 2025

Copy link
Copy Markdown

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