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

Adding attributes to enable printing the MOTD. #71

Merged
merged 3 commits into from
Feb 16, 2015

Conversation

dmerrick
Copy link
Contributor

We like having a MOTD, so I added these attributes to enable support for that.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 93.23% when pulling d996705 on IFTTT:adding_attributes_to_enable_motd into e071016 on TelekomLabs:master.

@dmerrick
Copy link
Contributor Author

Not sure why it failed the specs. Let me know if I can do anything to improve it!

@chris-rock
Copy link
Member

@dmerrick great work. Thanks for adding these options. One small improvement would be to add some test to ChefSpec: https://github.com/TelekomLabs/chef-ssh-hardening/blob/master/spec/recipes/server_spec.rb

I will have a look into the travis issues.

@chris-rock
Copy link
Member

@dmerrick could you rebase please? we fixed the tests in travis. Therefore everything should work now.

@dmerrick dmerrick force-pushed the adding_attributes_to_enable_motd branch from d996705 to 5192863 Compare February 9, 2015 19:56
Dana Merrick added 2 commits February 9, 2015 11:57
Personally, I think files should end in newlines, and I thought Rubocop
would enforce this...
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling a49e0da on IFTTT:adding_attributes_to_enable_motd into 31b8fb7 on TelekomLabs:master.

chris-rock added a commit that referenced this pull request Feb 16, 2015
Adding attributes to enable printing the MOTD.
@chris-rock chris-rock merged commit 5afc4c9 into dev-sec:master Feb 16, 2015
@chris-rock
Copy link
Member

@dmerrick Thank you very much!

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.

None yet

3 participants