Pre-commit discussion

[Nubuki-all]

Discussion to iron out some kinks before sending a pull request.

[Nubuki-all]

This Enum is a bit stifling since some clients actually allow specifying the amount of selectable options

neonize/neonize/utils/enum.py

Lines 371 to 381 in fd805a6

	class VoteType(Enum):
	"""
	Enumeration of vote types.
	Attributes:
	MULTIPLE (int): Allows selecting multiple options in a poll.
	SINGLE (int): Allows selecting only a single option in a poll.
	"""
	MULTIPLE = 0
	SINGLE = 1

I'd revert the build_poll function to accepting an interger

[Nubuki-all]

@krypton-byte
Proposed addition to neonize.utils.log (necessary changes has been made to goneonize )

import ctypes
import logging

from ..proto.Neonize_pb2 import LogEntry

try:
    from colorlog import ColoredFormatter
except Exception:
    ColoredFormatter = None

log = logging.getLogger(__name__)

if ColoredFormatter:
    formatter = ColoredFormatter(
        "%(asctime)s.%(msecs)03d %(log_color)s[%(name)s %(levelname)s] - %(message)s%(reset)s",
        datefmt="%H:%M:%S",
        log_colors={
            "INFO": "cyan",
            "WARNING": "yellow",
            "ERROR": "red",
            "CRITICAL": "bold_red",
        },
    )
    stream_handler = logging.StreamHandler()
    stream_handler.setFormatter(formatter)
else:
    stream_handler = logging.StreamHandler()

logging.basicConfig(
    format="%(asctime)s.%(msecs)03d [%(name)s %(levelname)s] - %(message)s",
    datefmt="%H:%M:%S",
    level=logging.INFO,
    handlers=[stream_handler],
)

clientlogger = logging.getLogger("whatsmeow.Client")
dblogger = logging.getLogger("Whatsmeow.Database")


def log_whatsmeow(binary: int, size: int):
    log_msg = LogEntry.FromString(ctypes.string_at(binary, size))
    if log_msg.Name == "Client":
        log = clientlogger
    elif log_msg.Name == "Database":
        log = dblogger
    else:
        log = logging.getLogger(f"whatsmeow.{log_msg.Name}")
    getattr(log, log_msg.Level.lower())(log_msg.Message)

I was thinking that with this, log should no longer be relied on to set the log_level of whatsmeow and instead it should instead use the log_level the user defined in their scripts