Feature: allow array of pathes in validate_absolute_path #323
Conversation
…o extending rspec tests and documentation
…ath, also extending rspec tests and documentation" as I mixed up a file (validate_array)... grml. This reverts commit 5889df7.
…o extending rspec tests and documentation (now for real)
|
why don't we just remove the check if the number of args is > 0? in that case we wouldn't have to put our paths into arrays in the first place. |
|
In my use case, it would be a great relief to have the possibility to use an array, as I throw stuff inside the check from hiera directly. And I can't think of a way to convert an array into function args within Puppet DSL...? Why would you remove the check if # args > 0? This wouldn't change the behaviour of Puppet::Util.absolute_path to check on a String via reg. ex., or do I miss some understanding of Ruby special handling of function calls? (I'm not THAT familiar with Ruby...) |
|
Hi there, |
|
I'll close this PR for now and create a separated squashed commit and a new PR for that. |
Sorry for the reverted stuff in between, GitHub does not seem to allow me to exclude stuff from the PR once commited...