linebender / druid-widget-nursery

A place where Druid widgets come to mature before moving to the Druid repo.
Apache License 2.0
88 stars 41 forks source link

Add a derive macro for Prism #101

Closed jplatte closed 2 years ago

jplatte commented 2 years ago

cc @xarvic 🙂

jplatte commented 2 years ago

That was the last force-push, testing whether existing where clauses or duplicate Clone predicates cause any issues (they don't seem to!).

jplatte commented 2 years ago

We should make sure we use the same versions, so users don't have to build them twice.

Cargo.toml dep versions aren't exact by degault. Virtually everything uses syn & quote 1.x.y, so only one will be compiled.

richard-uk1 commented 2 years ago

We should make sure we use the same versions, so users don't have to build them twice.

Cargo.toml dep versions aren't exact by degault. Virtually everything uses syn & quote 1.x.y, so only one will be compiled.

Yes sorry was more just thinking out loud. I don't expect it to be an issue.

jplatte commented 2 years ago

Fixed the panic message and did some other refactorings, ready for another review :)

xarvic commented 2 years ago

This is awesome! @jplatte, thank you so much :)

jplatte commented 2 years ago

@Maan2003 can this be merged? :)

maan2003 commented 2 years ago

Sure!