News-At-Command-Line [Enhancement] Migrate to Python3 ?

This would be great in Python 3, I think this would be appreciated.

Feb 04 '18 06:02 ctrlsam

cool. Go ahead

Feb 04 '18 07:02 Griffintaur

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

Feb 05 '18 07:02 ctrlsam

@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.

Feb 05 '18 09:02 ctrlsam

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

Feb 18 '18 22:02 TomFaulkner

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

Feb 19 '18 00:02 ctrlsam

I was thinking of doing the same.

Aside from Python 3, I saw you did some cleanup, did you do anything with filenames and pathing?

On Sun, Feb 18, 2018, 6:44 PM Sam Redmond [email protected] wrote:

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

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Griffintaur/News-At-Command-Line/issues/26#issuecomment-366564004, or mute the thread https://github.com/notifications/unsubscribe-auth/AOQ4TL757EEjziSg7dh05qFYMOnehBZGks5tWMPzgaJpZM4R4dgt .

Feb 19 '18 05:02 TomFaulkner

@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.

Feb 19 '18 07:02 ctrlsam

@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

Feb 19 '18 07:02 ctrlsam

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.

Feb 19 '18 15:02 TomFaulkner

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

Feb 20 '18 04:02 TomFaulkner

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

Feb 21 '18 14:02 TomFaulkner