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.
    • 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