search

Cretezy / pymessenger2

A Python Wrapper for the FB Messenger Bot API (Send/Receive API)
MIT License
25 stars 14 forks source link

Drop Python 2 support #4

Open sils opened 7 years ago

sils commented 7 years ago

Does anyone mind if we do that?

Cretezy commented 7 years ago

We could. Projects using this library will most likely be new and started using Python 3, so dropping 2 wouldn't affect much users. Do you want to make a pull request for it.

sils commented 7 years ago

Ok so for now we have only one branch for python 2 (dict.items vs iteritems) and dropping it isn't worth it but being open to this decision is good and when it gets more complicated we can do it.

Cretezy commented 7 years ago

Is this done?

sils commented 7 years ago

I didn't actively drop python 2 support but I also might have introduced some python 3 code unknowingly, if we actively want to drop this we can put an assert somewhere and clean up the little Python 2 code which is around to give people a good reason not to start writing their bot in python 2.

madEng84 commented 6 years ago

Oh no... I'm ready to use your chatbot for my python2.7 project. I will tell you if i'm encountering any problem in the integration. Elsewhere i will try to port my big django project in python 3+

Cretezy commented 6 years ago

Alright well let's try to keep Python 2 instead!

madEng84 commented 6 years ago

Thanks, i hope i will help you quickly :D