- Author: Adrien Thebo <email@example.com>
- Company: Puppet Labs
- Github ID: adrienthebo
- Pull Request 637 Discussion
- Pull Request 637 File Diff
This pull request removes the special casing of the empty string that was scattered across Facter. The main reason that this seems to have been added is that this:
Would behave correctly. This pull request removes the special casing from command execution and facts and restricts the special behavior to when Resolutions are evaluating a string. The existing behavior of `setcode 'foo'` will continue to work but the rest of the code base won't have to deal with `Facter::Core::Execution.exec` randomly returning `nil`.