Skip to content
This repository has been archived by the owner on Feb 9, 2024. It is now read-only.

Python 3 Support #36

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

mikey-
Copy link

@mikey- mikey- commented Mar 30, 2020

It's been a while since someone made a PR for this, so I figured since I was looking into it, I'd give it a crack...

Updated dependency versions, updated print statements and changed
iteration iteration operations on various dicts (in the creation
of option_defaults for example) in order for the module to run
on python 3.7.something.

There have been no attempts to improve much else, since that is not my
goal.

cheeeerrrrrzzzzzzzz and thank you

Updated dependency versions, updated print statements and changed
iteration iteration operations on various dicts (in the creation
of option_defaults for example) in order for the module to run
on python 3.7.something.

There have been no attempts to improve much else, since that is not my
goal, though it is evident that much work is required to bring the
module to a state by which it could be said to "work reliably".

To me, getting the module to run is enough; effort is better spent
creating a cdk module or somethin' idk byeeeeeeeeeeeeee
@mikey-
Copy link
Author

mikey- commented Jan 6, 2021

@thesamet is there anything stopping this from being merged?

@thesamet
Copy link
Contributor

thesamet commented Jan 7, 2021

Hi @mikey- , I'm no longer with TrueAccord so I am unable to merge. I am not sure if this project is still in use.

@mikey-
Copy link
Author

mikey- commented Jan 9, 2021

@thesamet fair enough :) Thanks for getting back to me anyway. I'll continue to use my fork I suppose.

@mikey-
Copy link
Author

mikey- commented Apr 11, 2021

@aplsms is this project still in use?

@mikey- mikey- changed the title DogPush doesn't run on python 3 Python 3 Support Aug 3, 2021
@mikey-
Copy link
Author

mikey- commented Jan 12, 2023

Thanks @terrafying :)

@moimikey is this something you can approve also?

@mikey-
Copy link
Author

mikey- commented Apr 12, 2023

@moimikey do you have any issues with the changes here?

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

Successfully merging this pull request may close these issues.

3 participants