FAForever Forums
    • Categories
    • Recent
    • Tags
    • Popular
    • Users
    • Groups
    • Login

    Where my python coders at?

    Scheduled Pinned Locked Moved Contribution
    24 Posts 7 Posters 2.0k Views 1 Watching
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • K Offline
      Katharsas
      last edited by

      Btw. when you open https://github.com/FAForever/server and then press dot key, you get IDE in browser.

      1 Reply Last reply Reply Quote 0
      • A Offline
        Aulex
        last edited by

        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.

        1 Reply Last reply Reply Quote 1
        • AskaholicA Offline
          Askaholic
          last edited by Askaholic

          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

          1 Reply Last reply Reply Quote 0
          • AskaholicA Offline
            Askaholic
            last edited by

            @KaletheQuick @CheeseBerry @Aulex Could you guys send me your GitHub names so I can add you as reviewers?

            KaletheQuickK 1 Reply Last reply Reply Quote 0
            • CheeseBerryC Offline
              CheeseBerry
              last edited by

              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)

              1 Reply Last reply Reply Quote 0
              • A Offline
                Aulex
                last edited by

                I already have access https://github.com/cheyans

                1 Reply Last reply Reply Quote 0
                • KaletheQuickK Offline
                  KaletheQuick @Askaholic
                  last edited by

                  @askaholic yes, I am KaletheQuick.

                  Sorry for not getting to you sooner.

                  You must deceive the enemy, sometimes your allies, but you must always deceive yourself!

                  1 Reply Last reply Reply Quote 0
                  • E Offline
                    Exselsior
                    last edited by

                    @Askaholic Also no guarantees how often I will be able to at the moment, but you can add https://github.com/Exselsior100

                    1 Reply Last reply Reply Quote 0
                    • AskaholicA Offline
                      Askaholic
                      last edited by

                      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.

                      1 Reply Last reply Reply Quote 1
                      • AskaholicA Offline
                        Askaholic
                        last edited by

                        Invites have been sent. You'll need to accept them before GitHub will let me request you as reviewers.

                        KaletheQuickK CheeseBerryC 2 Replies Last reply Reply Quote 1
                        • KaletheQuickK Offline
                          KaletheQuick @Askaholic
                          last edited by

                          @askaholic Just accepted. Don't know how much assistance I will be, but I'll do my best and hopefully learn in the process ❤️

                          You must deceive the enemy, sometimes your allies, but you must always deceive yourself!

                          1 Reply Last reply Reply Quote 0
                          • CheeseBerryC Offline
                            CheeseBerry @Askaholic
                            last edited by

                            @askaholic
                            I may be blind, but I can't find an invite. I assume it should just be in the github notifications?

                            E 1 Reply Last reply Reply Quote 0
                            • E Offline
                              Exselsior @CheeseBerry
                              last edited by

                              @cheeseberry I got my invite from an email

                              1 Reply Last reply Reply Quote 0
                              • CheeseBerryC Offline
                                CheeseBerry
                                last edited by

                                jup it was in my mails, ty!

                                1 Reply Last reply Reply Quote 1
                                • AskaholicA Offline
                                  Askaholic
                                  last edited by

                                  @KaletheQuick @CheeseBerry @Exselsior Please check your email, I requested y’all for a review. Also thanks to @Aulex for reviewing already.

                                  1 Reply Last reply Reply Quote 0
                                  • AskaholicA Offline
                                    Askaholic
                                    last edited by

                                    @KaletheQuick @CheeseBerry @Exselsior @Aulex (or anyone else) review away!

                                    https://github.com/FAForever/server/pull/867
                                    https://github.com/FAForever/server/pull/828

                                    KaletheQuickK 1 Reply Last reply Reply Quote 0
                                    • KaletheQuickK Offline
                                      KaletheQuick @Askaholic
                                      last edited by

                                      @askaholic alright. I actually have a day off next week, and have been brushing up on my python 🙂

                                      You must deceive the enemy, sometimes your allies, but you must always deceive yourself!

                                      1 Reply Last reply Reply Quote 1
                                      • AskaholicA Offline
                                        Askaholic
                                        last edited by

                                        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.

                                        1 Reply Last reply Reply Quote 3

                                        Hello! It looks like you're interested in this conversation, but you don't have an account yet.

                                        Getting fed up of having to scroll through the same posts each visit? When you register for an account, you'll always come back to exactly where you were before, and choose to be notified of new replies (either via email, or push notification). You'll also be able to save bookmarks and upvote posts to show your appreciation to other community members.

                                        With your input, this post could be even better 💗

                                        Register Login
                                        • First post
                                          Last post