avoid instantiating a git2 repo just to see if it is bare.

Instead, postpone the work-tree check to when it is needed and exit
with the same message.

Author: Byron

src/info/mod.rs

@@ -168,7 +168,7 @@ impl Info {
         let repo = git::discover(&config.input)?;
         let workdir = repo
             .work_dir()
-            .context("a non-bare repository is needed")?
+            .context("please run onefetch inside of a non-bare git repository")?
             .to_owned();
 
         let pending_changes = std::thread::spawn({

src/info/repo.rs

@@ -4,12 +4,10 @@ use crate::info::head_refs::HeadRefs;
 use anyhow::{Context, Result};
 use byte_unit::Byte;
 use git::bstr::BString;
-use git2::{Repository, RepositoryOpenFlags, Status, StatusOptions, StatusShow};
+use git2::{Status, StatusOptions, StatusShow};
 use git_repository as git;
 use git_repository::bstr::ByteSlice;
 use std::collections::HashMap;
-use std::path::Path;
-use std::path::PathBuf;
 use time::format_description::well_known::Rfc3339;
 use time::OffsetDateTime;
 
@@ -299,11 +297,6 @@ where
     dt.into().format(&Rfc3339).unwrap()
 }
 
-pub fn is_valid(repo_path: &PathBuf) -> Result<bool> {
-    let repo = Repository::open_ext(repo_path, RepositoryOpenFlags::empty(), Vec::<&Path>::new());
-    Ok(!repo?.is_bare())
-}
-
 #[cfg(test)]
 mod tests {
     use super::*;

src/main.rs

@@ -1,9 +1,9 @@
 #![cfg_attr(feature = "fail-on-deprecated", deny(deprecated))]
 
-use anyhow::{bail, Result};
+use anyhow::Result;
 use clap::{CommandFactory, Parser};
 use cli::Config;
-use info::{repo, Info};
+use info::Info;
 use std::io;
 use ui::printer::Printer;
 
@@ -27,12 +27,7 @@ fn main() -> Result<()> {
         return Ok(());
     }
 
-    if !repo::is_valid(&config.input)? {
-        bail!("please run onefetch inside of a non-bare git repository");
-    }
-
     let info = Info::new(&config)?;
-
     let mut printer = Printer::new(io::BufWriter::new(io::stdout()), info, config)?;
 
     printer.print()?;