Merge pull request #317 from llamahunter/fix-github-sync

Fix logic error preventing github integration.

ide/tests/test_find_project_root.py

@@ -103,3 +103,12 @@ def test_ignore_invalid_package_file(self):
             "src/main.c",
             "appinfo.json"
         ], "", "appinfo.json")
+
+    def test_PR_317(self):
+        """ PR 317 fixes a bug where find_project_root would fail with 11 character filenames """
+        self.run_test([
+            "MAINTAINERS",
+            "package.json",
+            "src/"
+            "src/main.c",
+        ], "", "package.json")

ide/utils/project.py

@@ -56,6 +56,8 @@ def find_project_root_and_manifest(project_items):
         # Check if the file is one of the kinds of manifest file
         for name in MANIFEST_KINDS:
             dir_end = base_dir.rfind(name)
+            if dir_end == -1:
+                continue
             # Ensure that the file is actually a manifest file
             if dir_end + len(name) == len(base_dir):
                 if is_manifest(name, item.read()):