Merge pull request #875 from douglasjacobsen/fix-includes-trailing-slash

Allow trailing slashes in workspace includes
GoogleCloudPlatform · Feb 11, 2025 · 10e9904 · 10e9904
2 parents c095dfa + 918d3ce
commit 10e9904
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/lib/ramble/ramble/workspace/workspace.py b/lib/ramble/ramble/workspace/workspace.py
@@ -1953,7 +1953,12 @@ def included_config_scopes(self):
         # highest-precedence scopes are last.
         includes = config_dict(self.config_sections["workspace"]["yaml"]).get("include", [])
         missing = []
-        for config_path in reversed(includes):
+        for full_config_path in reversed(includes):
+            # Remove trailing slash
+            config_path = full_config_path
+            if full_config_path.endswith("/"):
+                config_path = full_config_path[:-1]
+
             # allow paths to contain ramble config/environment variables, etc.
             config_path = substitute_path_variables(
                 config_path, local_replacements=self.workspace_paths()