Merge pull request #63 from CloudCannon/fix/checks-ordering

Use BTreeMap instead of HashMap for checks file to order the output

CHANGELOG.md

@@ -9,6 +9,8 @@
 
 ## Unreleased
 
+* Ensures the JSON output of the check command is sorted to avoid changes in git between runs
+
 ## v2.2.0 (August 15, 2024)
 
 * Adds the ability to build the default language to the root path, rather than placing it under a language code.

rosey/src/runners/checker.rs

@@ -1,6 +1,6 @@
 use serde::{Deserialize, Serialize};
 use std::{
-    collections::{BTreeMap, HashMap},
+    collections::BTreeMap,
     fs::{create_dir_all, read_to_string, File},
     io::{BufWriter, Write},
     str::FromStr,
@@ -10,7 +10,7 @@ use globwalk::DirEntry;
 
 use crate::{RoseyLocale, RoseyOptions, RoseyTranslation};
 
-#[derive(Serialize, Deserialize, Hash, PartialEq, Eq)]
+#[derive(Serialize, Deserialize, Hash, PartialEq, Eq, PartialOrd, Ord)]
 #[serde(rename_all = "camelCase")]
 enum RoseyCheckStates {
     Current,
@@ -25,8 +25,8 @@ struct RoseyCheck {
     current: bool,
     base_total: i32,
     total: i32,
-    states: HashMap<RoseyCheckStates, i32>,
-    keys: HashMap<String, RoseyCheckStates>,
+    states: BTreeMap<RoseyCheckStates, i32>,
+    keys: BTreeMap<String, RoseyCheckStates>,
 }
 
 pub struct RoseyChecker {