Merge pull request #83 from zombocom/schneems/absolute-with-contents

Expand `with_contents_dir` to absolute path
zombocom · Nov 28, 2024 · e1eab47 · e1eab47
2 parents 5a0c1c6 + 6413296
commit e1eab47
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,7 @@
 ## HEAD
 
+- Fix: `rundoc <file> --with-contents <dir>` now expands the path passed in to the `--with-contents` so relative paths can safely be used (https://github.com/zombocom/rundoc/pull/83)
+
 ## 3.1.1
 
 - Fix: Include all code sections in the event of a failure, not just the last one (https://github.com/zombocom/rundoc/pull/71)

diff --git a/lib/rundoc/cli_argument_parser.rb b/lib/rundoc/cli_argument_parser.rb
@@ -111,7 +111,7 @@ def call
         end
 
         opts.on("--with-contents <dir>", "Copies contents of directory into the tmp working dir") do |v|
-          options[:with_contents_dir] = v
+          options[:with_contents_dir] = File.expand_path(v)
         end
 
         opts.on("--on-success-dir <dir>", "Success dir, relative to CWD. i.e. `<rundoc.md/dir>/#{CLI::DEFAULTS::ON_SUCCESS_DIR}/`.") do |v|