🐛 command can now accept Optional[T] as a type

[ooliver1]

Changes

• adds: ...
• fixed: Support for Optional[...] in CommandArg #302
• improvements: ...

Check off the following

• I have tested my changes with the current requirements
• My Code follows the pep8 code style.

Test code

from os import getenv
from time import perf_counter
from typing import Annotated, Optional

from dotenv import load_dotenv

from pincer import Client
from pincer.commands import Description, command
from pincer.objects import MessageContext

marker = perf_counter()
load_dotenv("./dev/.env")


class Bot(Client):
    @Client.event
    async def on_ready(self):
        print(
            f"Logged in as {client.bot} after {perf_counter() - marker} seconds"
        )

    @command(guild=802586580766162964)
    async def cmd(
        self,
        ctx: MessageContext,
        content: Annotated[Optional[str], Description["..."]] = None,
    ) -> str:
        return f"{ctx.author.mention} h: {content}"


token = getenv("TOKEN")
assert token is not None
client = Bot(token)
client.run()

[codecov]

Codecov Report

Merging #480 (58ad6e5) into main (96bddae) will decrease coverage by 0.05%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #480      +/-   ##
==========================================
- Coverage   64.83%   64.78%   -0.06%     
==========================================
  Files         184      184              
  Lines        5984     5989       +5     
==========================================
  Hits         3880     3880              
- Misses       2104     2109       +5     

Impacted Files	Coverage Δ
pincer/commands/commands.py	23.01% <0.00%> (-0.96%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 96bddae...58ad6e5. Read the comment docs.

[ooliver1]

there is no commands dir so i cant really do much about coverage

[Enderchief]

Can you update this with the latest code on the main branch. There won't be any conflict, I would just like to see this pass the latest workflows.

[ooliver1]

@Endercheif Done

[ooliver1]

I think

[Sigmanificient]

Nox is failling, did you ran black & flake8?