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

Porting to python3 #146

Open
ColumOD opened this issue Apr 30, 2020 · 5 comments
Open

Porting to python3 #146

ColumOD opened this issue Apr 30, 2020 · 5 comments

Comments

@ColumOD
Copy link

ColumOD commented Apr 30, 2020

Hi, is this project still active? I am willing to help you with the porting to python3 as I have good experience doing this from other projects.

We should first make the code work in both python2 and 3 before making the full switch to python3. It would be good to get this done along with the addition of some new features before soccer returns!

Let me know if you are interested and I can get working on this asap!

@Saturn
Copy link
Collaborator

Saturn commented May 1, 2020

Yes, it would be good if this worked in Python 3. I can't imagine it being too much work. Maybe some of the messy code will cause some pain.

Back when I was working with this I was always using Python 2.7 just out of habit.

Of course, this used to be more active back when it was just started but now it has stagnated. But new features are welcome along with any refactoring that could make the code tidier etc.

Thanks,

@ColumOD
Copy link
Author

ColumOD commented May 2, 2020

Just setting up this now and some of the unit tests are failing. I am sure it is because of the fact there are currently no live games or fixtures and these exceptions are uncaught. Would you be able to quickly confirm they are not passing for you also before I go any further? Everything else is working just fine.

Thanks

@Saturn
Copy link
Collaborator

Saturn commented May 2, 2020

I think a lot (all) are failing and erroring because the code has changed without care of updating the unit tests to accompany the code change.

At least it shouldn't matter that there are no live games. Ideally.

I'll continue to have a look at it anyway.

@Shivanirudh
Copy link

Hello, so I have been trying to set this up and work. A peculiar thing i noticed was that i could not get any standings for UEFA Champions League. All other leagues work, and other options like fixtures, work for CL as well. Would you be able to confirm if you have this issue as well? Also, might this be because of the Champions League being divided into multiple groups?

Thanks in advance.

@Saturn
Copy link
Collaborator

Saturn commented Oct 6, 2020

Hmm, CL standings does not work for me.

I can't rememeber if it did work in the past. It may not have ever had the CL standings, but just the fixtures.

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

No branches or pull requests

3 participants