| UX note: it looks like most configuration options for bolt.yaml use a hyphen "-" character for separating words in multi-word configuration setting names; not underscores "_". Besides being consistent with existing config options, use of a hyphen as a word-separator is aligned with newer command-line flags in Puppet, most regex implementations ideas of what is part of a word vs. characters that can separate words, as well as the general nogui pattern guidance which seems to be "use hyphens as a word separators, NOT underscores, except when there's a technical constraint that dictates otherwise." Admittedly the nogui guidance is focused on things like command-line flags, but I think it applies well to config files as well. Whatever configuration setting we use should use hyphens as word separators, not underscores. If we stick with Nick's original setting name, the setting name would therefore become puppetfile-install-path. Curious though... is this really a "breaking" change, or did we incorrectly implement this? Reading the r10k docs, it sounds like r10k always defaults to "modules", regardless of Puppet's modulepath setting. Is that right? The Puppetfile is documented to support a "moduledir" setting. If we create a bolt.yaml setting for this then, maybe it should just be called puppetfile-moduledir. https://github.com/puppetlabs/r10k/blob/master/doc/puppetfile.mkd#moduledir |
