-
Notifications
You must be signed in to change notification settings - Fork 20
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
Fixing issue #57 and #58 #59
Fixing issue #57 and #58 #59
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.
Congrats on opening your first pull request and thank you for taking the time to help improve Ansible-Lockdown!
Please join in the conversation happening on the Discord Server as well.
Signed-off-by: fortinm <[email protected]>
Signed-off-by: fortinm <[email protected]>
Signed-off-by: fortinm <[email protected]>
Signed-off-by: fortinm <[email protected]>
Signed-off-by: fortinm <[email protected]>
Signed-off-by: fortinm <[email protected]>
Signed-off-by: fortinm <[email protected]>
Signed-off-by: fortinm <[email protected]>
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.
Thank you @mfortin ! Appreciate your PR!
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.
Approved
Overall Review of Changes:
Fixing issues identified below
Issue Fixes:
#57
#58
Enhancements:
Windows Server Core detection
Cloud Detection only applies to Azure
How has this been tested?:
N/A