diff options
| author | David Lönnhager <david.l@mullvad.net> | 2025-03-05 11:51:50 +0100 |
|---|---|---|
| committer | David Lönnhager <david.l@mullvad.net> | 2025-03-05 23:33:01 +0100 |
| commit | 111ee4282a641245e2b5877aecda18d5937eef28 (patch) | |
| tree | c9a6b1c18b0ff190d5baf1a442781f916d3f51fd /mullvad-update/src | |
| parent | e5063f68cdef8cb19a27296631b064d1a96fac07 (diff) | |
| download | mullvadvpn-111ee4282a641245e2b5877aecda18d5937eef28.tar.xz mullvadvpn-111ee4282a641245e2b5877aecda18d5937eef28.zip | |
Clarify meaning of default rollout
Diffstat (limited to 'mullvad-update/src')
| -rw-r--r-- | mullvad-update/src/format/mod.rs | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/mullvad-update/src/format/mod.rs b/mullvad-update/src/format/mod.rs index 82b4f16ea0..6829374022 100644 --- a/mullvad-update/src/format/mod.rs +++ b/mullvad-update/src/format/mod.rs @@ -63,18 +63,18 @@ pub struct Release { /// Installer details for different architectures pub installers: Vec<Installer>, /// Fraction of users that should receive the new version - #[serde(default = "default_rollout")] - #[serde(skip_serializing_if = "is_default_rollout")] + #[serde(default = "complete_rollout")] + #[serde(skip_serializing_if = "is_complete_rollout")] pub rollout: f32, } -/// By default, rollout includes all users -fn default_rollout() -> f32 { +/// A full rollout includes all users +fn complete_rollout() -> f32 { 1. } -fn is_default_rollout(b: impl std::borrow::Borrow<f32>) -> bool { - (b.borrow() - default_rollout()).abs() < f32::EPSILON +fn is_complete_rollout(b: impl std::borrow::Borrow<f32>) -> bool { + (b.borrow() - complete_rollout()).abs() < f32::EPSILON } /// App installer @@ -124,7 +124,7 @@ mod test { version: "2024.1".parse().unwrap(), changelog: "".to_owned(), installers: vec![], - rollout: default_rollout(), + rollout: complete_rollout(), }) .unwrap(); |
