Skip to content
This repository has been archived by the owner on Jan 5, 2024. It is now read-only.

Agenda for Public Core Meeting - 2016-05-24 19:00 UTC #100

Closed
sivel opened this issue May 20, 2016 · 1 comment
Closed

Agenda for Public Core Meeting - 2016-05-24 19:00 UTC #100

sivel opened this issue May 20, 2016 · 1 comment

Comments

@sivel
Copy link
Member

sivel commented May 20, 2016

Please leave a comment regarding any agenda item you wish to discuss. If you don't show up for the meeting, your item will be skipped.

If your IRC nick is different from your Github username, leave that as well.

@sivel
Copy link
Member Author

sivel commented May 20, 2016

Discuss making module requirement of including a function called main(). There is no explicit mention that a module must include a call to main(), to call it a requirement, let's ensure we have consensus, and then explicitly document it.

There is a benefit to requiring it, rather than just guarding the code to execute under if __name__ == "__main__":, in that ansible-testing can import the module (mocking any missing imports) and invoke main so that AnsibleModule is executed. Then we can gather the full argument_spec as passed to AnsibleModule and do doc/arg_spec comparisons for things where required/default are specified, and to ensure docs match the arg_spec.

@sivel sivel closed this as completed May 26, 2016
@dagwieers dagwieers added the core label Jun 24, 2017
@ansible ansible locked and limited conversation to collaborators Jul 19, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants