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 [192.168.1.2] (unknown [101.128.125.123]) by gnuweeb.org (Postfix) with ESMTPSA id 6318D80908; Fri, 21 Oct 2022 14:05:00 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gnuweeb.org; s=default; t=1666361101; bh=NbP7H+smxWaB4WkxwOjwXI313kwFQZzLmVnWmvCEIgg=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=awvULh7NmBrpb2zwA+Xkv2Vvdmktw4DF4F6KrCkmS9MiAN7BTowphrtakvFqW/TD8 jQ2DzQDn6J77T1M+8jNjDo0eRPTN7qPd9beVm2cqwLlE39igEV43o0vW6O5GhLPTSx uPsJABRAkVj4DYA8lHA4/FdGD3l0j9P3lGnzu4hJtMjl1+azHdJgvV3JXmH9WXpowg bW5bUjBEuY7n/iuq0De+pOpCnSCVyKJHFknr6IcQUuQ09Ve4yQwAPMEmHp4H4ftLax g0Q3buQSid0VfKRPOi+uEoOLhJYa1y8BYQhj8WGElvWXWjjCDiFUsShsBodELmPlMQ ng7TW/oVJjXUw== Message-ID: <982b723b-5c22-cfe2-6fcf-d0584b8c142d@gnuweeb.org> Date: Fri, 21 Oct 2022 21:04:56 +0700 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.3.3 Subject: Re: [PATCH v3 8/9] enum: Add Platform enumeration Content-Language: en-US To: Ammar Faizi Cc: Alviro Iskandar Setiawan , GNU/Weeb Mailing List References: <20221021134520.701-1-kiizuha@gnuweeb.org> <20221021134520.701-9-kiizuha@gnuweeb.org> <6fb1d238-de6a-325c-8d17-6801ae24c717@gnuweeb.org> From: Muhammad Rizki In-Reply-To: <6fb1d238-de6a-325c-8d17-6801ae24c717@gnuweeb.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit List-Id: On 21/10/2022 21.02, Ammar Faizi wrote: > On 10/21/22 8:45 PM, Muhammad Rizki wrote: >> --- /dev/null >> +++ b/daemon/enums/platform.py >> @@ -0,0 +1,7 @@ >> +import enum >> +from .base import EnumBase >> + >> + >> +class Platform(EnumBase): >> +    DISCORD = enum.auto() >> +    TELEGRAM = enum.auto() > > Why is this necessary to be auto generated enum? > > I don't see any reason of why we should use an extra library > here. I would just define them manually like: > >    DISCORD  = 1 >    TELEGRAM = 2 > > Thoughts? > You want so? But, I really like to use enum library here like everybody does.