Ideas for further checks

[kornicameister]

Here's a list of runtime thingies that caught us when working with loguru ;-).
Thanks to @Delgan support we know two of those being #18 and #7 . I bet there is more. This collective issue aims to bring this to our attention and have them implemented.

ToDo:

• [ ]

[issue-label-bot]

Issue-Label Bot is automatically applying the label feature_request to this issue, with a confidence of 0.82. Please mark this comment with 👍 or 👎 to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

[kornicameister]

@Delgan if you have something more feel free to let me know :). If there are some users already, let us know too :)

[Delgan]

There are different possible errors "at runtime", I guess we can figure out some of them by looking at the raise statements in the loguru sources.

Here are some examples of code causing exceptions but not detected by mypy / type-hints:

• logger.add(sys.stderr, level=-10) (negative level not allowed)
• logger.level("FOOBAR", no=-10) (negative level not allowed)
• logger.log(-10, "Message") (negative level not allowed)
• logger.level("INFO", no=111) (updating the no of an existing level is not allowed)
• logger.add("file.txt", foobar="xyz") (the open() builtin does not have a foobar argument)

There is also a whole range of "parsing" issues that I didn't listed (like parsing of color tags or rotation/retention).

[kornicameister]

Hmm...I think we can take some od those for later. @Delgan if you want to dive into wonderful world od mypy feel free to implement either of those 😁

[Delgan]

Honestly, I don't think it's worth spending time implementing this. I was just throwing some ideas. 😁

[kornicameister]

well perhaps having levels defined as int is enough. This is how typeshed does that. Maybe colors could be done like so if for example you requre hex like color codes, we could check against that. But that might low on prio for us.
I will keep this one here. If we people start using this plugin, they will be the best judge we can get ;)