You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'm surprised we haven't encountered this before now, probably because if someone wants to use serde_json::Value in their code deliberately then they probably already have serde_json in their dependencies.
The macros can emit code directly referencing
serde_json::Value
which may trigger compilation errors if the user doesn't have it in their own dependencies: https://discord.com/channels/665528275556106240/665528275556106243/969377568685981716I'm surprised we haven't encountered this before now, probably because if someone wants to use
serde_json::Value
in their code deliberately then they probably already haveserde_json
in their dependencies.Easy fix, though:
use
s topub use
: https://github.com/launchbadge/sqlx/blob/master/sqlx-core/src/types/json.rs#L4-L5JsonValue
,JsonRawValue
to this reexport: https://github.com/launchbadge/sqlx/blob/master/sqlx-core/src/types/mod.rs#L86serde_json::Value
tosqlx::types::JsonValue
in the following places:The text was updated successfully, but these errors were encountered: