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

Allow deny users to be managed from attributes #76

Merged
merged 1 commit into from Apr 10, 2015

Conversation

Projects
None yet
3 participants
@Rockstar04
Member

Rockstar04 commented Apr 9, 2015

This is a pull request to close #75.

Ruby is not my most comfortable language and I did quite some copy and pasting, but we pass all the rake tests.

I also tried to ensure the config file would be identical to the current one if the default attributes were used to prevent noise during chef runs after updating the whatever version this ends up in.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Apr 9, 2015

Coverage Status

Coverage remained the same at 100.0% when pulling 8a8425a on Rockstar04:allow-deny into 90400ec on TelekomLabs:master.

coveralls commented Apr 9, 2015

Coverage Status

Coverage remained the same at 100.0% when pulling 8a8425a on Rockstar04:allow-deny into 90400ec on TelekomLabs:master.

2 similar comments
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Apr 9, 2015

Coverage Status

Coverage remained the same at 100.0% when pulling 8a8425a on Rockstar04:allow-deny into 90400ec on TelekomLabs:master.

coveralls commented Apr 9, 2015

Coverage Status

Coverage remained the same at 100.0% when pulling 8a8425a on Rockstar04:allow-deny into 90400ec on TelekomLabs:master.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Apr 9, 2015

Coverage Status

Coverage remained the same at 100.0% when pulling 8a8425a on Rockstar04:allow-deny into 90400ec on TelekomLabs:master.

coveralls commented Apr 9, 2015

Coverage Status

Coverage remained the same at 100.0% when pulling 8a8425a on Rockstar04:allow-deny into 90400ec on TelekomLabs:master.

@chris-rock

This comment has been minimized.

Show comment
Hide comment
@chris-rock

chris-rock Apr 10, 2015

Member

@Rockstar04 Great work. I like it. Thank you very much.

Member

chris-rock commented Apr 10, 2015

@Rockstar04 Great work. I like it. Thank you very much.

chris-rock added a commit that referenced this pull request Apr 10, 2015

Merge pull request #76 from Rockstar04/allow-deny
Allow deny users to be managed from attributes

@chris-rock chris-rock merged commit 0ea9980 into dev-sec:master Apr 10, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment