-
Notifications
You must be signed in to change notification settings - Fork 120
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
Remove use of iteritems method #556
Conversation
Brings this module inline with requirements for Python 3
|
@syncrou 'carbonin' is an invalid assignee, ignoring... |
@billfitzgerald0120 would you be able to test this on a RHEL 8 appliance? |
Checked commit syncrou@ff7633e with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0 |
Pull Request Test Coverage Report for Build 3406
💛 - Coveralls |
Will this cause a problem with the ManageIQ appliance which is still running CentOS 7 (and thus python2)? |
Closing. Per @carbonin's comment. Will need to validate all changes are backwards compatible with Python 2 and 3. |
Closed in favor of #557 also test for backwards compatibility with Python 2 |
Brings this module inline with requirements for Python 3
BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1736116