1 Commits

Author SHA1 Message Date
102758c25c Release v0.3 2021-11-26 17:22:09 +01:00
2 changed files with 15 additions and 17 deletions

View File

@@ -949,6 +949,10 @@ pub fn run() {
.set_upstream(Some(&upstream_branch_name)) .set_upstream(Some(&upstream_branch_name))
.unwrap(); .unwrap();
} else { } else {
print_error(&format!(
"Remote branch {} not found",
&upstream_branch_name
));
let split_at = upstream_branch_name.find('/').unwrap_or(0); let split_at = upstream_branch_name.find('/').unwrap_or(0);
if split_at == 0 || split_at >= upstream_branch_name.len() - 1 { if split_at == 0 || split_at >= upstream_branch_name.len() - 1 {
print_error("Tracking branch needs to match the pattern <remote>/<branch_name>"); print_error("Tracking branch needs to match the pattern <remote>/<branch_name>");
@@ -993,10 +997,7 @@ pub fn run() {
); );
remote remote
.push(&[push_refspec], Some(&mut push_options)) .push(&[push_refspec], Some(&mut push_options))
.unwrap_or_else(|error| { .unwrap();
print_error(&format!("Pushing to {} ({}) failed: {}", remote_name, remote.url().unwrap(), error));
process::exit(1);
});
target_branch target_branch
.set_upstream(Some(&upstream_branch_name)) .set_upstream(Some(&upstream_branch_name))

View File

@@ -10,7 +10,6 @@ use crate::output::*;
pub enum RemoteType { pub enum RemoteType {
Ssh, Ssh,
Https, Https,
File,
} }
#[derive(Debug, PartialEq)] #[derive(Debug, PartialEq)]
@@ -128,14 +127,6 @@ mod tests {
); );
} }
#[test]
fn check_file_remote() {
assert_eq!(
detect_remote_type("file:///somedir"),
Some(RemoteType::File)
);
}
#[test] #[test]
fn check_invalid_remotes() { fn check_invalid_remotes() {
assert_eq!(detect_remote_type("https//example.com"), None); assert_eq!(detect_remote_type("https//example.com"), None);
@@ -156,6 +147,12 @@ mod tests {
fn check_unsupported_protocol_git() { fn check_unsupported_protocol_git() {
detect_remote_type("git://example.com"); detect_remote_type("git://example.com");
} }
#[test]
#[should_panic]
fn check_unsupported_protocol_file() {
detect_remote_type("file:///");
}
} }
pub fn detect_remote_type(remote_url: &str) -> Option<RemoteType> { pub fn detect_remote_type(remote_url: &str) -> Option<RemoteType> {
@@ -169,15 +166,15 @@ pub fn detect_remote_type(remote_url: &str) -> Option<RemoteType> {
if remote_url.starts_with("https://") { if remote_url.starts_with("https://") {
return Some(RemoteType::Https); return Some(RemoteType::Https);
} }
if remote_url.starts_with("file://") {
return Some(RemoteType::File);
}
if remote_url.starts_with("http://") { if remote_url.starts_with("http://") {
unimplemented!("Remotes using HTTP protocol are not supported"); unimplemented!("Remotes using HTTP protocol are not supported");
} }
if remote_url.starts_with("git://") { if remote_url.starts_with("git://") {
unimplemented!("Remotes using git protocol are not supported"); unimplemented!("Remotes using git protocol are not supported");
} }
if remote_url.starts_with("file://") || remote_url.starts_with('/') {
unimplemented!("Remotes using local protocol are not supported");
}
None None
} }
@@ -257,7 +254,7 @@ pub fn clone_repo(
&remote.url &remote.url
)); ));
match remote.remote_type { match remote.remote_type {
RemoteType::Https | RemoteType::File => { RemoteType::Https => {
let mut builder = git2::build::RepoBuilder::new(); let mut builder = git2::build::RepoBuilder::new();
builder.bare(is_worktree); builder.bare(is_worktree);
builder.clone(&remote.url, &clone_target)?; builder.clone(&remote.url, &clone_target)?;