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

Prepare for syn v2 #578

Merged
merged 6 commits into from
Apr 27, 2023
Merged

Prepare for syn v2 #578

merged 6 commits into from
Apr 27, 2023

Conversation

jonasbb
Copy link
Owner

@jonasbb jonasbb commented Mar 18, 2023

Update darling to v0.20 and syn to v2, require serde v1.0.157 (first using syn v2).
This moves all direct dependencies to crates using syn v2.

syn v2 has problems with the #[serde_as(as = "..")] style of attributes, since as is a keyword and keywords are not allowed in meta attributes. Instead pre-process the token stream to replace all as with r#as which can be parsed.

Bump thiserror and cxx in the lock file.
Not all crates are using syn v2 yet.

@codecov
Copy link

codecov bot commented Mar 21, 2023

Codecov Report

Merging #578 (008a62f) into master (74858ec) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #578   +/-   ##
=======================================
  Coverage   63.56%   63.56%           
=======================================
  Files          35       35           
  Lines        2072     2072           
=======================================
  Hits         1317     1317           
  Misses        755      755           
Impacted Files Coverage Δ
serde_with/src/flatten_maybe.rs 100.00% <ø> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Update serde to a version using syn v2.
Update cxx and thiserror in the lock file to versions using syn v2.
Update darling to v20
@jonasbb jonasbb marked this pull request as ready for review April 27, 2023 21:26
@jonasbb
Copy link
Owner Author

jonasbb commented Apr 27, 2023

bors r+

bors bot added a commit that referenced this pull request Apr 27, 2023
578: Prepare for syn v2 r=jonasbb a=jonasbb

Update darling to v0.20 and syn to v2, require serde v1.0.157 (first using syn v2).
This moves all direct dependencies to crates using syn v2.

syn v2 has problems with the `#[serde_as(as = "..")]` style of attributes, since `as` is a keyword and keywords are not allowed in meta attributes. Instead pre-process the token stream to replace all `as` with `r#as` which can be parsed.

Bump thiserror and cxx in the lock file.
Not all crates are using syn v2 yet.

Co-authored-by: Jonas Bushart <[email protected]>
@bors
Copy link
Contributor

bors bot commented Apr 27, 2023

Build failed:

@jonasbb
Copy link
Owner Author

jonasbb commented Apr 27, 2023

bors r+

@bors
Copy link
Contributor

bors bot commented Apr 27, 2023

Build succeeded:

@bors bors bot merged commit f95573c into master Apr 27, 2023
@bors bors bot deleted the prepare-syn-2 branch April 27, 2023 21:46
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.

1 participant