Skip to content

Comments

Update to syn 2 and darling 0.20.5#308

Merged
TedDriggs merged 4 commits intomasterfrom
syn-2
Feb 14, 2024
Merged

Update to syn 2 and darling 0.20.5#308
TedDriggs merged 4 commits intomasterfrom
syn-2

Conversation

@TedDriggs
Copy link
Collaborator

@TedDriggs TedDriggs commented Feb 12, 2024

This uses the most recent major version of syn, as well as the most recent version of darling.

Breaking Changes

  1. It's no longer possible to use type to override the type of a field; instead, ty must be used.

Fixes #292

- Bump crates to 2018 edition; this is needed for generated code to
  produce usable error messages.
- Bump darling dependency to 0.20.5
- Remove support for `field(type = "...")` and leave only `field(ty = "...")`
- Update compiletest outputs

Remove interim change
This makes click-to-definition work when reading source code
Copy link

@jtran jtran left a comment

Choose a reason for hiding this comment

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

Looks good to me. I built my project with this branch, and it looks good.

@TedDriggs
Copy link
Collaborator Author

I'm going to wait to merge this until TedDriggs/darling#263 has landed, as I may want to ship with a darling dependency of 0.20.6 instead of 0.20.5. However, that hopefully won't take long to resolve, and at that point this will be unblocked.

@TedDriggs TedDriggs merged commit 04bcde6 into master Feb 14, 2024
@TedDriggs TedDriggs deleted the syn-2 branch February 14, 2024 21:49
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.

Update syn dependency?

2 participants