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

    Where my python coders at?

    Scheduled Pinned Locked Moved Contribution
    24 Posts 7 Posters 1.8k 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.
    • 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
                            • First post
                              Last post