Skip to content
This repository was archived by the owner on Dec 26, 2020. It is now read-only.

Conversation

bachp
Copy link
Contributor

@bachp bachp commented Jun 30, 2017

No description provided.

@rndmh3ro rndmh3ro self-requested a review June 30, 2017 11:03
@rndmh3ro rndmh3ro self-assigned this Jun 30, 2017
Copy link
Member

@rndmh3ro rndmh3ro left a comment

Choose a reason for hiding this comment

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

Looks really good, thank you! I only like on minor thing to be added.

@@ -0,0 +1,3 @@
{% for key in ssh_server_revoked_keys %}
Copy link
Member

Choose a reason for hiding this comment

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

Can you add a # {{ansible_managed}} on the first line, please?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added. I also checked and the comment doesn't interfere with the functionality. 👍

@rndmh3ro rndmh3ro merged commit 5c68c65 into dev-sec:master Jun 30, 2017
@bachp bachp deleted the revoked-keys branch June 30, 2017 20:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants