-
Notifications
You must be signed in to change notification settings - Fork 62
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
Upstream this module? #72
Comments
I too would love to see this to be part of upstream ansible modules. @kwoodson if it's ok with you we could work on getting this merged. |
Just saw that you already started the process to get this merged (#38841).... |
@Gnitset @cpointner |
hi @kwoodson, i wrote a module myself and i found the process to be pretty straight forward. You usually just have to open the pull request fix some (linter) errors when ansibot returns that the module failed the tests and then, if nobody else sees a problem / missing required feature, you can request your module to be merged. TL;DR just fight ansibot and some reviewers until it becomes mergeable. If you tell me what current problems you're facing just tell me and I'll be happy to help you :) |
Any particular reason as to why this isn't submitted upstream to be a part of the core modules?
The text was updated successfully, but these errors were encountered: