Hi,

I think this is a great idea.

About the naming things problem: I don't know if this is overkill, but how
about having an exception like e.g. ReportableError or KnownError that
errors like using an invalid scope in your example can inherit from (e.g.
InvalidScope) can inherit from. All errors inheriting from that
ReportableError or KnownError could then be caught at the top level and be
reported differently without a noisy traceback (e.g. only printing out the
place where it occured and the error type and message).

Cheers,
Oliver


On Thu, 4 Oct 2018 at 15:51 Florian Bruhin <m...@the-compiler.org> wrote:

> Hey,
>
> On Thu, Oct 04, 2018 at 10:19:21AM -0300, Bruno Oliveira wrote:
> > This new exception type (I've used UsageError on my PR just for proof of
> > concept) should be part of the public API so plugins can also use it.
> >
> > I'm writing to the ML because it would be beneficial to get feedback
> from a
> > larger audience about what the *name* of the exception should be.
> >
> > UserError? InternalError?
>
> I dislike both, but without having a better idea (sorry :D). Here's why:
>
> UsageError is dangerously close to UsageError. It's also not clear
> from the name why they differ.
>
> InternalError sounds like the exceptions prefixed with INTERNALERROR>,
> (i.e. unhandled exceptions inside pytest), which seems like the exact
> opposite of what this is.
>
> Maybe we should rename UsageError to CliUsageError, or make sure
> UsageError is usable for this purpose?
>
> Florian
>
> --
> https://www.qutebrowser.org | m...@the-compiler.org (Mail/XMPP)
>    GPG: 916E B0C8 FD55 A072 | https://the-compiler.org/pubkey.asc
>          I love long mails! | https://email.is-not-s.ms/
> _______________________________________________
> pytest-dev mailing list
> pytest-dev@python.org
> https://mail.python.org/mailman/listinfo/pytest-dev
>
_______________________________________________
pytest-dev mailing list
pytest-dev@python.org
https://mail.python.org/mailman/listinfo/pytest-dev

Reply via email to