-
Notifications
You must be signed in to change notification settings - Fork 136
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
fix: rename var sshd -> sshd_config and debug output #299
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did not notice any issue. Maybe just a suggestion to provide some simple sanity test that the old name keeps working (ideally with both role
and include_role
syntax`).
+1 |
583a983
to
12b6130
Compare
12b6130
to
d6b4548
Compare
Thanks @Jakuje, you're a star 🌟 |
Enhancement:
Address role prefix on variable names.
Reason: Coding standards
Result:
Issue Tracker Tickets (Jira or BZ if any):
#297, #246