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

Add ability to specify a symlink for r10k.yaml #41

Merged
merged 3 commits into from
Jun 2, 2014

Conversation

ghoneycutt
Copy link
Member

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.

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>
Copy link
Collaborator

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

Copy link
Member Author

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?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@acidprime
Copy link
Collaborator

@ghoneycutt if you can flip the pre's back to GFM code fences then I will get this merged. Thanks for your contribution.
-Z

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.
@ghoneycutt
Copy link
Member Author

@acidprime No problem, rebased and squashed to change pre tags to code fences

acidprime added a commit that referenced this pull request Jun 2, 2014
Add ability to specify a symlink for r10k.yaml
@acidprime acidprime merged commit e8be417 into voxpupuli:master Jun 2, 2014
@ghoneycutt ghoneycutt deleted the add_symlink_for_config_file branch June 2, 2014 18:39
@ghoneycutt
Copy link
Member Author

@acidprime thanks for the merge. Could you please tag a release of the code that contains this?

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.

2 participants