[Enhancement] Migrate to Python3 ? about news-at-command-line HOT 11 OPEN

cool. Go ahead

from news-at-command-line.

@Griffintaur I have submitted a PR, do you think you can take a look at it, thanks

from news-at-command-line.

@Griffintaur depending on what you want, you might want to think about having the Python 2.7 version in a separate branch for those that don't use Python 3.

from news-at-command-line.

Is this going to get merged? I noticed there are several unmerged PRs and master hasn't been updated in months.

from news-at-command-line.

I know right, this is a little frustrating. If it is not merged after a week I'll make a separate repo.

from news-at-command-line.

from news-at-command-line.

@TomFaulkner I have not changed file names/paths. If it helps, you can fork my copy: https://github.com/SillySam/News-At-Command-Line

I will be happy to contribute to your copy as this guy doesn't seem to active.

PS: I am just about to make another change on my fork that removes the owner's file headers and email, etc.

from news-at-command-line.

@TomFaulkner I also want to note, when you make your own copy, please make your API key as I don't think the owner would like the key going viral :D

from news-at-command-line.

Yeah, that is what I had in mind. Since moving to Python 3, without maintaining 2.7 compatibility is a breaking change, I think it would be a good time to make other breaking changes for PEP8 and a more pythonic ways of doing things.

from news-at-command-line.

Posted this as a PR on @sillysam's fork. But, thought I would mention it here. I have a more Pythonic fork from @sillysam's fork. I cleaned up quite a bit, more still needs done, but it's posted at least. I plan to do more as I get time.

https://github.com/TomFaulkner/News-At-Command-Line

from news-at-command-line.

Since this was merged, I moved discussion to #30 so this issue can be closed.

from news-at-command-line.