Where my python coders at?
-
You normally don't have to touch any low level networking stuff. Typical server changes feel like "normal" python, so don't worry about that.
-
Btw. when you open https://github.com/FAForever/server and then press dot key, you get IDE in browser.
-
I did server python work professionally for a few years and still work with it today. I can take a look occasionally, no guarantees I’ll regularly be able to.
-
I just need to get peoples GitHub account names so I can add you to the list of people who I can request for review. So either send me that info or just go and do a review on one of the PR’s then I’ll be able to add you.
Either of these are my top priority right now.
https://github.com/FAForever/server/pull/825
https://github.com/FAForever/server/pull/830 -
@KaletheQuick @CheeseBerry @Aulex Could you guys send me your GitHub names so I can add you as reviewers?
-
I'm ChessBerry on GH: https://github.com/ChessBerry
(I don't know who the guys are that took both the name CheeseBerry and CheeseBerries last month but they certainly are annoying) -
I already have access https://github.com/cheyans
-
@askaholic yes, I am KaletheQuick.
Sorry for not getting to you sooner.
-
@Askaholic Also no guarantees how often I will be able to at the moment, but you can add https://github.com/Exselsior100
-
Great, thanks! No worries @Exselsior, I’m going to try something new which is to create a reviewers group and then request the whole group for review, so hopefully at least a few people from the group will be able to find time for a quick review.
-
Invites have been sent. You'll need to accept them before GitHub will let me request you as reviewers.
-
@askaholic Just accepted. Don't know how much assistance I will be, but I'll do my best and hopefully learn in the process
️ -
@askaholic
I may be blind, but I can't find an invite. I assume it should just be in the github notifications? -
@cheeseberry I got my invite from an email
-
jup it was in my mails, ty!
-
@KaletheQuick @CheeseBerry @Exselsior Please check your email, I requested y’all for a review. Also thanks to @Aulex for reviewing already.
-
@KaletheQuick @CheeseBerry @Exselsior @Aulex (or anyone else) review away!
https://github.com/FAForever/server/pull/867
https://github.com/FAForever/server/pull/828 -
@askaholic alright. I actually have a day off next week, and have been brushing up on my python
-
Need help from any and all who see this. This is a very important pr for making it so that people who frequently cause matchmaker games to fail will be prevented from queuing for a certain period of time.
https://github.com/FAForever/server/pull/885
It needs to be reviewed and tested. Hopefully I’ll find some time to organize this soonTM.
