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

    Where my python coders at?

    Scheduled Pinned Locked Moved Contribution
    24 Posts 7 Posters 2.6k Views
    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.
    • AskaholicA
      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
        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
          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
            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
              Exselsior @CheeseBerry
              last edited by

              @cheeseberry I got my invite from an email

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

                jup it was in my mails, ty!

                1 Reply Last reply Reply Quote 1
                • AskaholicA
                  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
                    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
                      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
                        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
                        • First post
                          Last post