First, its not a pep8 violation, having the import at the bottom is but that can be considered again as 'not a violation' as it improves redability when errors happen (line number line up).
2nd it is not a real import and is already confusing, I personally much preferred the << MODULES_BASIC >> way as it was clear that this is NOT a python import. Your change makes it even more misleading as it implies a restricted import which is not true. Last, but not least, we don't enforce PEP8 nor are particularly keen on following it blindly as many others do. The PEP8 spec itself tells you not do do so, that is why PEP8 only changes are not normally accepted. ---------- Brian Coca -- You received this message because you are subscribed to the Google Groups "Ansible Project" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/ansible-project/CACVha7cmpZjSDwyB4zq812DGao5yad-s7L%3DtpwwyFWYwjtvWMw%40mail.gmail.com. For more options, visit https://groups.google.com/d/optout.
