Add EOL at EOF for JSON files (#238)

butane_cli/src/lib.rs

@@ -44,7 +44,9 @@ impl CliState {
     pub fn save(&self, base_dir: &Path) -> Result<()> {
         let path = base_dir.join("clistate.json");
         let mut file = File::create(path)?;
-        file.write_all(serde_json::to_string_pretty(self)?.as_bytes())?;
+        let mut contents = serde_json::to_string_pretty(self)?;
+        contents.push('\n');
+        file.write_all(contents.as_bytes())?;
         Ok(())
     }
 }

butane_core/src/db/mod.rs

@@ -109,8 +109,9 @@ impl ConnectionSpec {
     pub fn save(&self, path: &Path) -> Result<()> {
         let path = conn_complete_if_dir(path);
         let mut f = fs::File::create(path)?;
-        f.write_all(serde_json::to_string_pretty(self)?.as_bytes())
-            .map_err(|e| e.into())
+        let mut contents = serde_json::to_string_pretty(self)?;
+        contents.push('\n');
+        f.write_all(contents.as_bytes()).map_err(|e| e.into())
     }
     /// Load a previously saved connection spec
     pub fn load(path: impl AsRef<Path>) -> Result<Self> {

butane_core/src/migrations/fsmigrations.rs

@@ -78,7 +78,9 @@ impl FsMigration {
     }
 
     fn write_info(&self, info: &MigrationInfo) -> Result<()> {
-        self.write_contents("info.json", serde_json::to_string_pretty(info)?.as_bytes())
+        let mut contents = serde_json::to_string_pretty(info)?;
+        contents.push('\n');
+        self.write_contents("info.json", contents.as_bytes())
     }
 
     fn write_sql(&self, name: &str, sql: &str) -> Result<()> {
@@ -380,8 +382,9 @@ impl FsMigrations {
         }
         let path = self.root.join("state.json");
         let mut f = self.fs.write(&path)?;
-        f.write_all(serde_json::to_string_pretty(state)?.as_bytes())
-            .map_err(|e| e.into())
+        let mut contents = serde_json::to_string_pretty(state)?;
+        contents.push('\n');
+        f.write_all(contents.as_bytes()).map_err(|e| e.into())
     }
     /// Detach the latest migration from the list of migrations,
     /// leaving the migration on the filesystem.

examples/getting_started/.butane/clistate.json

@@ -1,3 +1,3 @@
 {
-    "embedded": true
+  "embedded": true
 }