address PR review (see details)

* remove renovate group as it's not needed anymore
* repository discovery will open with isolation
This commit is contained in:
Sebastian Thiel 2024-03-18 15:10:08 +01:00
parent 9bf91497db
commit 363d2da187
No known key found for this signature in database
GPG key ID: 9CB5EE7895E8268B
2 changed files with 6 additions and 11 deletions

View file

@ -67,15 +67,6 @@
internalChecksFilter: 'strict',
groupName: 'msrv',
},
{
matchManagers: [
'cargo',
],
matchPackageNames: [
'gix',
],
groupName: 'gix',
},
// Goals:
// - Rollup safe upgrades to reduce CI runner load
// - Have lockfile and manifest in-sync (implicit rules)

View file

@ -263,8 +263,12 @@ impl<'gctx> PathSource<'gctx> {
/// Returns [`Some(gix::Repository)`](gix::Repository) if there is a sibling `Cargo.toml` and `.git`
/// directory; otherwise, the caller should fall back on full file list.
fn discover_gix_repo(&self, root: &Path) -> CargoResult<Option<gix::Repository>> {
let repo = match gix::discover(root) {
Ok(repo) => repo,
let mut mapping = gix::sec::trust::Mapping::default();
mapping.full = gix::open::Options::isolated();
mapping.reduced = gix::open::Options::isolated();
let repo = match gix::ThreadSafeRepository::discover_opts(root, Default::default(), mapping)
{
Ok(repo) => repo.to_thread_local(),
Err(e) => {
tracing::debug!(
"could not discover git repo at or above {}: {}",