-
Notifications
You must be signed in to change notification settings - Fork 68
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
Fixed the netaddr error msg for clarity #244
Conversation
Build succeeded. ✔️ ansible-galaxy-importer SUCCESS in 3m 59s |
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.
This should use missing_required_lib instead so we can have a consistent messaging for missing packages.
Build succeeded. ✔️ ansible-galaxy-importer SUCCESS in 3m 58s |
Build succeeded. ✔️ ansible-galaxy-importer SUCCESS in 3m 12s |
SUMMARY
Fixed the netaddr package error msg from:
The ipv4 filter requires python's netaddr be installed on the ansible controller
To:
The ipv4 filter requires that the Python netaddr package be installed on the ansible controller
For better clarity
Fixing #243
ISSUE TYPE
COMPONENT NAME
plugins/plugin_utils/base/ipaddr_utils.py
plugins/plugin_utils/base/ipaddress_utils.py