mab122
/
radicle-heartwood-lfs
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

cli: Don't strip initial message from editor 

This was problematic when the initial message was a valid default.
This commit is contained in:
Alexis Sellier 2023-03-22 16:33:52 +01:00
parent 63fc06c817
commit 6c1e2d0ddd
No known key found for this signature in database
2 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
radicle-cli/src/commands/patch/create.rs
View File

@ -40,7 +40,7 @@ pub fn handle_patch_message(
let (title, description) = (title.trim(), description.trim()); let (title, description) = (title.trim(), description.trim());
if title.is_empty() { if title.is_empty() {
anyhow::bail!("a title must be given"); anyhow::bail!("a patch title must be provided");
} }
Ok((title.to_string(), description.to_owned())) Ok((title.to_string(), description.to_owned()))

4
radicle-term/src/editor.rs
View File

@ -69,12 +69,10 @@ impl Editor {
process::Command::new(cmd).arg(&self.path).spawn()?.wait()?; process::Command::new(cmd).arg(&self.path).spawn()?.wait()?;
let text = fs::read_to_string(&self.path)?; let text = fs::read_to_string(&self.path)?;
let text = text.strip_prefix(&initial).unwrap_or(&text);
if text.trim().is_empty() { if text.trim().is_empty() {
return Ok(None); return Ok(None);
} }
Ok(Some(text.to_owned())) Ok(Some(text))
} }
} }