kemitix/git-next
1
1
Fork 1
Code Issues 17 Pull requests 1 Projects 1 Releases 41 Packages 2 Wiki Activity Actions

fix: don't modify config of external repos
Some checks failed
ci/woodpecker/push/cron-docker-builder Pipeline was successful
Details
ci/woodpecker/push/push-next Pipeline was successful
Details
ci/woodpecker/push/tag-created Pipeline was successful
Details
Rust / build (push) Failing after 1m35s
Details

Browse source 
The git config files of external repos are read-only.

This is the only place where we make reference to a remote named
'origin', so this also closes kemitix/git-next#85.

Closes kemitix/git-next#85
This commit is contained in:
Paul Campbell 2024-07-12 08:12:45 +01:00
parent 5f36282667
commit 024775faa5
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
crates/git/src/repo_details.rs
View file

@ -138,7 +138,7 @@ impl RepoDetails {
#[tracing::instrument] #[tracing::instrument]
pub fn write_remote_url(&self, url: &RemoteUrl) -> Result<(), kxio::fs::Error> { pub fn write_remote_url(&self, url: &RemoteUrl) -> Result<(), kxio::fs::Error> {
if self.gitdir.storage_path_type() != StoragePathType::Internal { if self.gitdir.storage_path_type() != StoragePathType::Internal {
// return Err(Not an internal repo) return Ok(());
} }
let fs = self.gitdir.as_fs(); let fs = self.gitdir.as_fs();
// load config file // load config file