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
Python 2 to 3 updates #5528
Python 2 to 3 updates #5528
Conversation
4ea9ebd
to
38840bd
Compare
you'll need to backout changes to tools/c7n_mailer to pass ci, or add additional unit tests to that package, per #5521 mailer has had low test coverage historically, to effect movement towards resolving that, changes to that package typically require adding additional tests. |
Hey @CMeza99 I should have the code cov up in the next day or so if you want to wait on it. Will message when it's resolved. |
Thank you @kapilt and @JohnHillegass ! |
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.
lgtm, thanks
Some quick Py3 updates:
import __future__