-
-
Notifications
You must be signed in to change notification settings - Fork 167
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
Add ability to specify a symlink for r10k.yaml #41
Add ability to specify a symlink for r10k.yaml #41
Conversation
This patch allows me to have a symlink for r10k.yaml that points to where the config file is located. This is useful if you want to put the config file in a non-standard location while not having to tell r10k to look in a different place for the config file.
```puppet | ||
include r10k::prerun_command | ||
``` | ||
<pre> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any reason for not using the code fencing here, as even the forge now supports it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there any difference between three backticks and a pre tag?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
code fences support syntax highlighting
https://help.github.com/articles/github-flavored-markdown#syntax-highlighting
@ghoneycutt if you can flip the pre's back to GFM code fences then I will get this merged. Thanks for your contribution. |
This patch allows you to parameterize the inclusion of r10k::prerun_command and keep you from having to include a separate class. The code optionally does an include on the class in order to maintain backward compatibility. Recommend that in a future release the code from the prerun_command class is moved into the r10k class where it is currently being included.
@acidprime No problem, rebased and squashed to change pre tags to code fences |
Add ability to specify a symlink for r10k.yaml
@acidprime thanks for the merge. Could you please tag a release of the code that contains this? |
This patch allows me to have a symlink for r10k.yaml that points to
where the config file is located. This is useful if you want to put the
config file in a non-standard location while not having to tell r10k to
look in a different place for the config file.