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

chore: update dependencies #491

Closed
wants to merge 4 commits into from
Closed

Conversation

BradLugo
Copy link
Contributor

No description provided.

@@ -108,6 +108,7 @@ pub fn validated_type(input: TokenStream) -> TokenStream {
#validated_type_ident,
::validation::ValidationError
> {
panic!("here");
Copy link
Contributor

Choose a reason for hiding this comment

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

is this debugging code? :)

Copy link
Contributor Author

@BradLugo BradLugo Jan 12, 2024

Choose a reason for hiding this comment

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

Yeah 😞 There may be a few instances of this before the PR is marked ready. Most of the advice I found when searching how to debug macro issues was along the lines of "just add debug statements until you figure it out."

If anyone has a better way to debug macros, please let me know.

Copy link
Contributor

Choose a reason for hiding this comment

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

no, that's about how i do it.

no worries about giving up on some of the packages. we can do those one-by-one.

dmah42
dmah42 previously approved these changes Jan 14, 2024
@dmah42
Copy link
Contributor

dmah42 commented Jan 14, 2024

merged in some other PRs and i think i got the merge right here. hopefully it doesn't make things harder for you.

@dmah42
Copy link
Contributor

dmah42 commented Jan 18, 2024

manually merged Cargo.lock :/

@mccormickt mccormickt mentioned this pull request Jan 20, 2024
@dmah42
Copy link
Contributor

dmah42 commented Jul 8, 2024

we just did a big dependency update in #496 so i'm going to close this in deference to that.

@dmah42 dmah42 closed this Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants