fix(get-modified-packages): fix a bug with top-level project reposito…

…ries (#61) * fix: get-modified-packages.sh with less diff Signed-off-by: h-ohta <[email protected]> * apply review Signed-off-by: Kenji Miyake <[email protected]> Co-authored-by: Kenji Miyake <[email protected]>
kenji-miyake · Feb 7, 2022 · e7a38ea · e7a38ea
1 parent 349f213
commit e7a38ea
Showing 1 changed file with 6 additions and 7 deletions.
diff --git a/get-modified-packages/get-modified-packages.sh b/get-modified-packages/get-modified-packages.sh
@@ -28,13 +28,6 @@ function find_package_dir() {
 
     target_dir=$(dirname "$1")
     while true; do
-        parent_dir=$(dirname "$target_dir")
-
-        # Exit if no parent found
-        if [ "$parent_dir" = "$target_dir" ]; then
-            return 0
-        fi
-
         # Output package name if package.xml found
         if [ -f "$target_dir/package.xml" ]; then
             if [ ! -f "$target_dir/COLCON_IGNORE" ]; then
@@ -43,6 +36,12 @@ function find_package_dir() {
             fi
         fi
 
+        # Exit if no parent found
+        parent_dir=$(dirname "$target_dir")
+        if [ "$parent_dir" = "$target_dir" ]; then
+            return 0
+        fi
+
         # Move to parent dir
         target_dir=$parent_dir
     done