NabDev/NabBot

Consider simplifying this complex logical expression.

Open

        if a < 125 or (r < 15 and g < 15 and b < 15) or (r > 254 and g > 254 and b > 254):
            continue
        return r, g, b

Avoid deeply nested control flow statements.

Open

                        with closing(wiki_db.cursor()) as c:
                            c.execute("""SELECT item.name FROM item
                                         LEFT JOIN item_attribute att on item_id = item.article_id
                                         WHERE item.name LIKE ? AND article_id = item_id AND att.name = 'imbuement'
                                         LIMIT 1""", (item,))

Avoid deeply nested control flow statements.

Open

                            if not image_copy.getpixel((x + xs, y + ys)) == slot_border[s] \
                                and image_copy.getpixel((x + xs, y + ys)) not in [(24, 24, 24, 255),
                                                                                  (55, 55, 55, 255),
                                                                                  (57, 57, 57, 255),
                                                                                  (75, 76, 76, 255),

Avoid deeply nested control flow statements.

Open

                            if last_scan_ss >= current_ss:
                                log.debug(f"{tag} {values[0].name} | {values[1].name} | Already saved")
                                await asyncio.sleep(0.1)
                                continue
                        try:

Avoid deeply nested control flow statements.

Open

                        if result:
                            has_marketable = True
                            emoji = "💎"
                        else:
                            emoji = ""

Avoid too many return statements within this function.

Open

        return "moments"

Avoid too many return statements within this function.

Open

        return "{0} minutes".format(minutes)

Avoid too many return statements within this function.

Open

    return commands.check(predicate)

Avoid too many return statements within this function.

Open

    return commands.check(predicate)

Avoid too many return statements within this function.

Open

    return None

Avoid too many return statements within this function.

Open

        return "1 hour"

Avoid too many return statements within this function.

Open

        return "none"

Refactor this function to reduce its Cognitive Complexity from 26 to the 15 allowed.

Open

def check_sql_database(conn: sqlite3.Connection):

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

Refactor this function to reduce its Cognitive Complexity from 16 to the 15 allowed.

Open

    async def stats(self, ctx: NabCtx, *, params: str):

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check