Maciej Szulik added the comment:

> Sorry. I didn't get you. You mean, same tests for PR body, right?

Yeah, exactly that. I'd like to see both body and title tests. Sorry I was 
initially noting down my thoughts and apparently forgot to "normalize" that 
part of the sentence ;)


> I was thinking of adding "Exception, e" instead of checking for just
> KeyError in dispatch method. But that will also catch Unauthorized,
> MethodNotAllowed and unsupportedmediatype. Can you suggest a better way to
> do this?

Only going through the code and figure out what are the possible exceptions
and just name them explicitly. No magic option here, sorry :(

_______________________________________________________
PSF Meta Tracker <[email protected]>
<http://psf.upfronthosting.co.za/roundup/meta/issue589>
_______________________________________________________
_______________________________________________
Tracker-discuss mailing list
[email protected]
https://mail.python.org/mailman/listinfo/tracker-discuss
Code of Conduct: https://www.python.org/psf/codeofconduct/

Reply via email to