From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on gnuweeb.org X-Spam-Level: X-Spam-Status: No, score=-1.8 required=5.0 tests=ALL_TRUSTED,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,NO_DNS_FOR_FROM, URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 Received: from [172.16.0.2] (unknown [8.34.202.23]) by gnuweeb.org (Postfix) with ESMTPSA id 24BDE80BE8; Sun, 11 Sep 2022 13:16:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gnuweeb.org; s=default; t=1662902180; bh=gDDlIFFEhROrk4oy1vqWWLZmLtQig9weYw6dYbajaoY=; h=Date:To:Cc:References:From:Subject:In-Reply-To:From; b=ODWoBTQU4A2g0ns2AGaAjlt9ICWZOIU7BV+xklRf58HZlL3tjacYxLSzOx++tBykz avRVtpTI4zmWfbcvkvFoU8qyvLSN0tk3Kt2nTH/4p7JEzszwwv8D8H9tn7nG4TOdY/ uEnr2zNr5pqvg2YHzu1wF2Zic+BG4y37ppFI+HCDjV0DwzJIiKSeURYKS35RDuJ1JV hdfC5GpBmIGqMNal3tUkk+LWaXvR3anB8sPX1PMH5vtWGs4MjtZg0m1cTWvjkjvsiE A34bz3S0VqVM9GBJVn/vxPOPGjWC5MSbL9fM/uC+KHo7USy+Rmb0teceR39pzuqFNy 6au73Ch8JcstA== Message-ID: Date: Sun, 11 Sep 2022 20:16:12 +0700 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.11.0 Content-Language: en-US To: Muhammad Rizki Cc: Alviro Iskandar Setiawan , GNU/Weeb Mailing List References: <20220911103323.1949-1-kiizuha@gnuweeb.org> <20220911103323.1949-2-kiizuha@gnuweeb.org> From: Ammar Faizi Subject: Re: [RFC PATCH v1 01/17] [telegram] Move the Telegram bot source code In-Reply-To: <20220911103323.1949-2-kiizuha@gnuweeb.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit List-Id: On 9/11/22 5:33 PM, Muhammad Rizki wrote: > --- a/.gitignore > +++ b/.gitignore > @@ -140,4 +140,5 @@ data.json > *.patch > > # configuration file > -daemon/config.py > +daemon/telegram/config.py > +daemon/dscord/config.py Overall, I love this patch, but I hate the dscord part here. At this point, we don't have dscord file yet. Please move the dscord ignore entry into the patch that adds Discord files. With that fixed, you can append this in the next version of this patch: Acked-by: Ammar Faizi -- Ammar Faizi