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

deps: Update syn to 2.0 #961

Closed
wants to merge 5 commits into from
Closed

deps: Update syn to 2.0 #961

wants to merge 5 commits into from

Conversation

GrayJack
Copy link
Contributor

@GrayJack GrayJack commented May 20, 2024

The tests failed to me locally. Both without and with this modifications

@emilio
Copy link
Collaborator

emilio commented May 28, 2024

The tests failed to me locally. Both without and with this modifications

Can you elaborate on how? CI looks green here tho, so that's always something :)

@emilio
Copy link
Collaborator

emilio commented May 28, 2024

I fixed / simplified some of the va-list handling (which otherwise breaks with #970).

@emilio emilio closed this in aa8ea65 May 28, 2024
@emilio
Copy link
Collaborator

emilio commented May 28, 2024

Thanks!

@emilio
Copy link
Collaborator

emilio commented May 28, 2024

Also if you could file an issue about the tests failing on your machine, it'd be great to get it sorted out.

@GrayJack
Copy link
Contributor Author

GrayJack commented May 28, 2024

Can you elaborate on how? CI looks green here tho, so that's always something :)

It was the first comment, I fixed the failures in the following commits and forgot to update the comment 😅

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.

2 participants