fix: handle package removal failures

lib/cmds/remove.sh

@@ -65,7 +65,11 @@ function remove_pkg_repo {
 
 	# Remove package from git, then from DB, them commit the change
 	if db::package_exists "$pkg"; then
-		git::remove_submodule "$pkg"
+		if ! git::remove_submodule "$pkg"; then
+			error "Failed to remove $pkg locally. Please check the logs!"
+			return 1
+		fi
+
 		db::remove_package "$pkg"
 		git::commit ":fire: remove $pkg"
 		removed=1

lib/git.sh

@@ -65,11 +65,12 @@ function git::remove_submodule {
 
 		# Remove the git submodule completely
 		git submodule deinit -f "$pkg_name"
-		git rm -rf "$pkg_name"
 		rm -rf ".git/modules/$pkg_name"
 
-		# Remove the package locally
-		rm -rf "$pkg_name"
+		# Remove from local & from git -- although not strictly necessary, return statement is here to
+		# make explicit that we rely upon the return code of this call to decide whether this entire
+		# command succeded or failed
+		git rm -rf "$pkg_name" || return 1
 	) > /dev/null
 }