Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FOR REVIEW: proposed file path additions for facter #79

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ahpook
Copy link

@ahpook ahpook commented Jul 8, 2016

Updates the file_paths spec to include proposed locations
for new config file and fact cache locations for facter.
Follow along at puppetlabs/puppet-rfc

Updates the file_paths spec to include proposed locations
for new config file and fact cache locations for facter.
Follow along at puppetlabs/puppet-rfc
@MikaelSmith
Copy link
Contributor

@Magisus have you taken a look at this? Not sure how the conf.d fits into current implementation plans.

@Magisus
Copy link
Contributor

Magisus commented Aug 31, 2016

This supports what I have implemented so far. I haven't yet referred to either the cache directory or the config snippets directory in my code. I know in theory HOCON supports including entire other files into a top-level config, but I'm not sure if that has been fully implemented in cpp-hocon yet. This would be a good test case for that functionality. What is an example use case?

@MikaelSmith
Copy link
Contributor

We've released this. Were there any changes, or is it ready to merge?

@Magisus
Copy link
Contributor

Magisus commented May 8, 2017

We do not have support for conf.d yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants