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

Avoid duplicate resource names because of warnings #127

Merged
merged 1 commit into from Oct 28, 2016

Conversation

Projects
None yet
3 participants
@artem-sidorenko
Member

artem-sidorenko commented Oct 25, 2016

Cloning resource attributes for directory[/etc/ssh] from prior resource (CHEF-3694)
Previous directory[/etc/ssh]: /tmp/kitchen/cache/cookbooks/ssh-hardening/recipes/server.rb:56:in `from_file'
Current  directory[/etc/ssh]: /tmp/kitchen/cache/cookbooks/ssh-hardening/recipes/client.rb:26:in `from_file' at 1 location:
Avoid duplicate resource names because of warnings
Cloning resource attributes for directory[/etc/ssh] from prior resource (CHEF-3694)
Previous directory[/etc/ssh]: /tmp/kitchen/cache/cookbooks/ssh-hardening/recipes/server.rb:56:in `from_file'
Current  directory[/etc/ssh]: /tmp/kitchen/cache/cookbooks/ssh-hardening/recipes/client.rb:26:in `from_file' at 1 location:

Signed-off-by: Artem Sidorenko <artem@posteo.de>
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Oct 25, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling 8e74a14 on artem-sidorenko:warnings into b8f5a06 on dev-sec:master.

coveralls commented Oct 25, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling 8e74a14 on artem-sidorenko:warnings into b8f5a06 on dev-sec:master.

@atomic111

This comment has been minimized.

Show comment
Hide comment
@atomic111

atomic111 Oct 28, 2016

Member

@artem-sidorenko thank you for the improvement

Member

atomic111 commented Oct 28, 2016

@artem-sidorenko thank you for the improvement

@atomic111 atomic111 merged commit 6c33405 into dev-sec:master Oct 28, 2016

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 100.0%
Details

@artem-sidorenko artem-sidorenko deleted the artem-forks:warnings branch Nov 23, 2016

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