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

(maint) Puppet style guide improvements #8

Merged
merged 1 commit into from Feb 8, 2012

Conversation

Projects
None yet
2 participants
@kelseyhightower

kelseyhightower commented Feb 8, 2012

Before this patch, defined resources do not live in their own manifests
which is a violation of the Puppet style guide.

This patch resolves this issue by moving all defined resources to their
own Puppet manifests.

This patch also resolves issue #5: Bad path for 'ln' command by updating
the exec resource to include /bin to the list of paths.

Kelsey Hightower
(maint) Puppet style guide improvements
Before this patch, defined resources do not live in their own manifests
which is a violation of the Puppet style guide.

This patch resolves this issue by moving all defined resources to their
own Puppet manifests.

This patch also resolves issue #5: Bad path for 'ln' command by updating
the exec resource to include `/bin` to the list of paths.

BenoitCattie added a commit that referenced this pull request Feb 8, 2012

Merge pull request #8 from kelseyhightower/maint/master/style_guide_f…
…ixes

(maint) Puppet style guide improvements

@BenoitCattie BenoitCattie merged commit 4d9d696 into BenoitCattie:master Feb 8, 2012

@BenoitCattie

This comment has been minimized.

Show comment
Hide comment
@BenoitCattie

BenoitCattie Feb 8, 2012

Owner

Thanks

Owner

BenoitCattie commented Feb 8, 2012

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment