Write rust-version with a newline and trim it when loading it

Author: Kobzol

src/bin/rustc_josh_sync.rs

@@ -125,6 +125,7 @@ fn load_context(config_path: &Path, rust_version_path: &Path) -> anyhow::Result<
         .context("cannot load config. Run the `init` command to initialize it.")?;
     let rust_version = std::fs::read_to_string(&rust_version_path)
         .inspect_err(|err| eprintln!("Cannot load rust-version file: {err:?}"))
+        .map(|version| version.trim().to_string())
         .map(Some)
         .unwrap_or_default();
     Ok(SyncContext {

src/sync.rs

@@ -86,7 +86,11 @@ impl GitSync {
         // We pass `--no-verify` to avoid running git hooks.
         // We do this before the merge so that if there are merge conflicts, we have
         // the right rust-version file while resolving them.
-        std::fs::write(&self.context.last_upstream_sha_path, &upstream_sha).with_context(|| {
+        std::fs::write(
+            &self.context.last_upstream_sha_path,
+            &format!("{upstream_sha}\n"),
+        )
+        .with_context(|| {
             anyhow::anyhow!(
                 "cannot write upstream SHA to {}",
                 self.context.last_upstream_sha_path.display()