-
Notifications
You must be signed in to change notification settings - Fork 165
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
Putting DatagenProvider
behind a feature
#4800
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good start. Still needs some work to make CI pass.
provider/datagen/src/transform/cldr/locale_canonicalizer/likely_subtags.rs
Outdated
Show resolved
Hide resolved
provider/datagen/src/transform/cldr/locale_canonicalizer/likely_subtags.rs
Show resolved
Hide resolved
@@ -32,7 +32,7 @@ pub struct NoFallbackOptions {} | |||
/// the locale fallback algorithm. If internal fallback is requested for an exporter that does | |||
/// not support it, an error will occur. | |||
#[non_exhaustive] | |||
#[derive(Debug, Copy, Clone, PartialEq, Eq, serde::Deserialize, serde::Serialize, Hash)] | |||
#[derive(Debug, Copy, Clone, PartialEq, Eq, Hash)] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thought: I guess removing these impls is technically a breaking change. I might be okay with it though
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes this is breaking, this would be the only reason for the driver crate to need serde...
#4721