kemitix/git-next
1
1
Fork 1
Code Issues 18 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 10s
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 c20de45ff7
2 changed files with 2 additions and 2 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

2
crates/repo-actor/src/tests/handlers/validate_repo.rs
View file

@ -354,7 +354,7 @@ async fn should_reject_message_with_expired_token() -> TestResult {
} }
#[test_log::test(actix::test)] #[test_log::test(actix::test)]
// NOTE: failed then passed on retry: count = 3 // NOTE: failed then passed on retry: count = 5
async fn should_send_validate_repo_when_retryable_error() -> TestResult { async fn should_send_validate_repo_when_retryable_error() -> TestResult {
//given //given
let fs = given::a_filesystem(); let fs = given::a_filesystem();