Skip to content

Conversation

@tadeohepperle
Copy link
Contributor

@tadeohepperle tadeohepperle commented Feb 28, 2024

Close #1054

| syn::GenericArgument::Constraint(_)
| syn::GenericArgument::Const(_) => {}
syn::GenericArgument::AssocType(arg) => self.visit_type(&arg.ty),
_ => {}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is needed now because the enum became non_exhaustive


/// Traverse syntax tree.
fn visit_type_param_bound(&mut self, bound: &syn::TypeParamBound) {
match bound {
Copy link
Contributor

Choose a reason for hiding this comment

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

you use if let here to make clippy happy

Copy link
Contributor

@niklasad1 niklasad1 left a comment

Choose a reason for hiding this comment

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

LGTM

@tadeohepperle tadeohepperle marked this pull request as ready for review February 28, 2024 18:16
@tadeohepperle tadeohepperle requested a review from a team as a code owner February 28, 2024 18:16
@tadeohepperle tadeohepperle merged commit 3f16132 into master Feb 29, 2024
@tadeohepperle tadeohepperle deleted the tadeohepperle/syn-2.0 branch February 29, 2024 10:00
@niklasad1 niklasad1 mentioned this pull request Feb 29, 2024
@lexnv lexnv mentioned this pull request Mar 5, 2024
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.

Upgrade to Syn 2.0

4 participants