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
refactor(turborepo): Bumping biome crates and moving to derive for deserialize #7875
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
4 Ignored Deployments
|
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @NicholasLYang and the rest of your teammates on Graphite |
🟢 Turbopack Benchmark CI successful 🟢Thanks |
dd3b49e
to
3985bdb
Compare
b7f3bef
to
a6db7a8
Compare
🟢 CI successful 🟢Thanks |
3985bdb
to
65f5017
Compare
beffe28
to
01edaf8
Compare
01edaf8
to
83c9b29
Compare
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.
Guide
#[serde(rename_all = "camelCase")] | ||
pub struct SpacesJson { | ||
pub id: Option<UnescapedString>, | ||
#[serde(flatten)] | ||
pub other: Option<serde_json::Value>, |
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.
I don't see this being used anywhere?
text: Option<Arc<str>>, | ||
#[serde(skip)] | ||
path: Option<Arc<str>>, | ||
span: Spanned<()>, |
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.
Had to change this because previous types wouldn't work well with derive.
#[serde(skip_serializing_if = "Spanned::is_none")] | ||
cache: Spanned<Option<bool>>, | ||
#[serde(skip_serializing_if = "Option::is_none")] | ||
cache: Option<Spanned<bool>>, |
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.
Same as above. Plays well with derive
@@ -802,6 +804,7 @@ mod tests { | |||
&root_package_json, | |||
false, | |||
)?; | |||
|
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.
|
||
struct ConfigurationOptionsVisitor; | ||
|
||
impl DeserializationVisitor for ConfigurationOptionsVisitor { |
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.
Turns out we don't ever even use this!
@@ -199,7 +199,7 @@ mod tests { | |||
|
|||
#[test_case(Spanned { value: 10, range: Some(0..2), path: None, text: None }, "10")] | |||
#[test_case(Spanned { value: "hello world", range: None, path: None, text: Some(Arc::from("hello world")) }, "\"hello world\"")] | |||
#[test_case(Spanned { value: json!({ "name": "George", "age": 100 }), range: None, path: None, text: Some(Arc::from("hello world")) }, "{\"age\":100,\"name\":\"George\"}")] | |||
#[test_case(Spanned { value: json!({ "name": "George", "age": 100 }), range: None, path: None, text: Some(Arc::from("hello world")) }, "{\"name\":\"George\",\"age\":100}")] |
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.
Huh, was serde sorting too?
f66be69
to
841ad0f
Compare
841ad0f
to
9d035d9
Compare
Description
Biome shipped a new derive for their
Deserializable
trait. This saves us from having to manually implement the trait, barring a few exceptions.Testing Instructions
Closes TURBO-2740