Tweak migrations template fn versions_matches

hulthe · hulthe · commit 436f2ca873c3 · 2024-02-28T11:06:02.000+01:00
Make it not take a mut ref to the json value
diff --git a/mullvad-daemon/src/migrations/vX.rs.template b/mullvad-daemon/src/migrations/vX.rs.template
@@ -26,7 +26,7 @@ pub fn migrate(settings: &mut serde_json::Value) -> Result<()> {
     Ok(())
 }
 
-fn version_matches(settings: &mut serde_json::Value) -> bool {
+fn version_matches(settings: &serde_json::Value) -> bool {
   settings
       .get("settings_version")
       // TODO
@@ -37,7 +37,6 @@ fn version_matches(settings: &mut serde_json::Value) -> bool {
 #[cfg(test)]
 mod test {
     use super::{migrate, version_matches};
-    use serde_json;
 
     // TODO: Implement tests. Look at other migration modules for inspiration.
 }

Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ pub fn migrate(settings: &mut serde_json::Value) -> Result<()> {`
`26`	`26`	`Ok(())`
`27`	`27`	`}`
`28`	`28`
`29`		`-fn version_matches(settings: &mut serde_json::Value) -> bool {`
	`29`	`+fn version_matches(settings: &serde_json::Value) -> bool {`
`30`	`30`	`settings`
`31`	`31`	`.get("settings_version")`
`32`	`32`	`// TODO`
`@@ -37,7 +37,6 @@ fn version_matches(settings: &mut serde_json::Value) -> bool {`
`37`	`37`	`#[cfg(test)]`
`38`	`38`	`mod test {`
`39`	`39`	`use super::{migrate, version_matches};`
`40`		`- use serde_json;`
`41`	`40`
`42`	`41`	`// TODO: Implement tests. Look at other migration modules for inspiration.`
`43`	`42`	`}`