Skip to content

Conversation

@ajwerner
Copy link
Contributor

Fixes #522

Copy link
Member

@tomhoule tomhoule left a comment

Choose a reason for hiding this comment

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

Good catch. Thanks for the fix!

@tomhoule tomhoule merged commit 5f8c172 into graphql-rust:main Feb 11, 2025
10 of 11 checks passed
@danieleades
Copy link
Contributor

this was merged with failing CI jobs, will need to be fixed in a new PR to get the builds passing again

@Sytten
Copy link
Contributor

Sytten commented Jun 27, 2025

This broke a bunch of stuff and needs to be reverted or improved:

`?` operator cannot convert from `std::string::String` to `std::vec::Vec<std::string::String>`
expected struct `std::vec::Vec<std::string::String>`
   found struct `std::string::String`
= note: `?` operator cannot convert from `std::string::String` to `std::option::Option<std::vec::Vec<std::string::String>>`
   = note: expected enum `std::option::Option<std::vec::Vec<std::string::String>>`
            found struct `std::string::String`

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.

codegen: breaking change for ID! types

4 participants