From 618a14bb159d657c7461a2cb1c3b531d17cde730 Mon Sep 17 00:00:00 2001
From: Mikito Takada <mikito.takada@gmail.com>
Date: Sun, 16 Oct 2016 13:08:53 -0700
Subject: [PATCH] Update readme.md

---
 README.md                   | 1 +
 lib/command-requirements.js | 5 ++---
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/README.md b/README.md
index 5b3d413..e7ec08e 100644
--- a/README.md
+++ b/README.md
@@ -12,6 +12,7 @@
 ## Changelog
 
 - Looking for more committers! Let me know if you're interested; gr currently meets my fairly limited needs but I know it can become even more useful given a larger core team and/or a stronger vision around how it can support usage in a team working on multiple repos.
+- `0.5.2`: `gr` now handles git submodules better, thanks @cojomojo!
 - `0.5.1`: `gr discover` now handles paths outside the home directory as expected, thanks @farmerchris!
 - `0.5.0`: `gr discover` is now an alias for `gr tag discover`; `gr discover` now accepts path arguments, shows progress during a scan and only scans five levels deep by default. Thanks @coderjoe for the patches!
 - `0.4.1`: `gr status` now only invokes git once per directory, thanks @coderjoe!
diff --git a/lib/command-requirements.js b/lib/command-requirements.js
index 2365773..e22f78c 100644
--- a/lib/command-requirements.js
+++ b/lib/command-requirements.js
@@ -12,8 +12,7 @@ module.exports = {
     } catch (e) {
       // check if the directory is a submodule
       // addresses https://github.com/mixu/gr/issues/54
-      if (e.code === 'ENOTDIR' || e.code === 'ENOENT')
-      { 
+      if (e.code === 'ENOTDIR' || e.code === 'ENOENT') {
         var parentPath = path.dirname(req.path);
         try {
           stat = fs.statSync(parentPath + '/.gitmodules');
@@ -33,10 +32,10 @@ module.exports = {
     try {
       stat = fs.statSync(req.path + '/Makefile');
       if (stat.isFile()) {
+        return true;
       }
     } catch (e) { }
     if (req.format === 'human') {
-        return true;
       console.log('Skipped ' + req.path + ' as it does not have a ./Makefile file.');
     }
     return false;