-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
add azure_rm_keyvaultkey_info module #55754
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.
in general looks good, but I think we shouldn't merge _facts any longer but _info, so we should wait until this is established.
@yungezz, just so you are aware we have a dedicated Working Group for azure. |
@yungezz existing conflicting files, Please resolve when you're available! Thanks! |
The test
The test
The test
The test
|
The test
The test
The test
The test
The test
The test
The test
The test
The test
The test
|
@yungezz Please finished PR update when you're free! Thank you very much! |
The test
The test
The test
|
@yungezz Please help finished the PR changed when you're free, and push for review. Thanks a lot! |
The test
The test
The test
|
ready_for_review |
@JunyiYi @Madhura-CSI @brusMX @caohai @cperrin88 @devigned @gustavomcarmo @haroldwongms @iphilpot @jeking3 @julienstroheker @lmazuel @needgithubid @obsoleted @smile37773 @sozercan @techknowlogick @trstringer @yaweiw @yuwzho As a maintainer of a module in the same namespace this new module has been submitted to, your vote counts for shipits. Please review this module and add |
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.
looks good
SUMMARY
add azure_rm_keyvaultkey_info module
ISSUE TYPE
COMPONENT NAME
ADDITIONAL INFORMATION