tokio-rs / valuable

MIT License
185 stars 19 forks source link

Introduce derive macro attribute processing (Take 2) #130

Closed johnhurt closed 2 weeks ago

johnhurt commented 3 weeks ago

This pr includes all of the changes from @taiki-e 's pr from several years ago: #75. It looks like that pr lost traction, so I am hoping to give it new life because I would like to use the features it provides as well at the one new attribute I am introducing.

The original pr included the entire mechanism for processing derive-macro attributes, and utilizing them to allow users to

My only contribution to this is:

I am happy to restructure or add to this pr to help it get incorporated it faster. I'm not sure why the original pr wasn't merged, but it still seems like a very useful feature. Thanks!

taiki-e commented 3 weeks ago

Thanks for the PR.

I honestly have no intention of reviewing this PR as is. The reason is that you compressed everything into one commit, which makes it very difficult to review the specific diff with my PR and requires the entire review to be done from the beginning.