lib/licensed/git.rb in licensed-1.3.4 vs lib/licensed/git.rb in licensed-1.4.0

@@ -5,33 +5,41 @@
       # Returns whether git commands are available
       def available?
         @git ||= Licensed::Shell.tool_available?("git")
       end
 
+      def git_repo?
+        return false unless available?
+        !Licensed::Shell.execute("git", "status", allow_failure: true).empty?
+      end
+
+      # Returns the root of the current git repository
+      # or nil if not in a git repository.
       def repository_root
-        return unless available?
-        @root ||= Pathname.new(Licensed::Shell.execute("git", "rev-parse", "--show-toplevel"))
+        return unless git_repo?
+        Licensed::Shell.execute("git", "rev-parse", "--show-toplevel")
       end
 
       # Returns the most recent git SHA for a file or directory
       # or nil if SHA is not available
       #
       # descriptor - file or directory to retrieve latest SHA for
       def version(descriptor)
-        return unless available? && descriptor
+        return unless git_repo? && descriptor
         Licensed::Shell.execute("git", "rev-list", "-1", "HEAD", "--", descriptor, allow_failure: true)
       end
 
       # Returns the commit date for the provided SHA as a timestamp
       #
       # sha - commit sha to retrieve date
       def commit_date(sha)
-        return unless available? && sha
+        return unless git_repo? && sha
         Licensed::Shell.execute("git", "show", "-s", "-1", "--format=%ct", sha)
       end
 
+      # Returns the files in the git repository from `git ls-files --recurse-submodules`
       def files
-        return unless available?
+        return unless git_repo?
         output = Licensed::Shell.execute("git", "ls-files", "--full-name", "--recurse-submodules")
         output.lines.map(&:strip)
       end
     end
   end