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

changes for Python 3 compatibility #91

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

hiroshitoda
Copy link

@hiroshitoda hiroshitoda commented Jun 8, 2020

including some changes to use collectd-cloudwatch in Python 3 environment like below:

  • library name
  • encoding/decoding around str type and bytes type

after merging this PR, we can't use this in Python 2 environment.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@hiroshitoda
Copy link
Author

resolved conflictions at 6fadf9a

@aghassemlouei
Copy link

Anything that we can do to help merge this into master?

@archoversight
Copy link

This is affecting us too, would love to see this resolved. Python 2 was EOL January 1st 2020.

@berenddeboer
Copy link

This PR isn't immediately useful as it uses the easy_install command, no clue what that is.

@berenddeboer
Copy link

OK, to get that easy_install working, make sure pip is installed on your server.

@nickzakharov
Copy link

Any update on this? Would love to be able to use this plugin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants