Skip to content
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

Paultaiton 20210409 requirements doc #485

Conversation

paultaiton
Copy link
Contributor

SUMMARY

Change documentation for doc_fragments/azure.py to more explicitly mention requirements-azure.txt as well as point to full install instructions.

Fixes #186 , but that's a very old issue that was recently referenced.

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

doc_fragment/azure.py

ADDITIONAL INFORMATION

The online documentation for the collection's modules doesn't sufficiently indicate the full requirements necessary for executing modules. See https://docs.ansible.com/ansible/latest/collections/azure/azcollection/azure_rm_storageaccount_module.html . This PR is submitted in the attempt to give better indication that there are more requirements than what are displayed per module and in the auto-added "Note" banner that Ansible project adds to azcollection docs, and to point new users to the ansible-galaxy readme for azcollection.


@Fred-sun Fred-sun added documentation-pr Improvements or additions to documentation medium_priority Medium priority ready_for_review The PR has been modified and can be reviewed and merged labels Apr 12, 2021
@haiyuazhang haiyuazhang merged commit 33742ab into ansible-collections:dev Apr 12, 2021
@paultaiton
Copy link
Contributor Author

Thanks @Fred-sun and @haiyuazhang , appreciate the quick review and merge.

@paultaiton paultaiton deleted the paultaiton_20210409_requirements-doc branch April 12, 2021 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation-pr Improvements or additions to documentation medium_priority Medium priority ready_for_review The PR has been modified and can be reviewed and merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ERROR! Unexpected Exception, this is probably a bug: No module named 'msrest'
3 participants