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 <metatrac...@psf.upfronthosting.co.za> <http://psf.upfronthosting.co.za/roundup/meta/issue589> _______________________________________________________ _______________________________________________ Tracker-discuss mailing list Tracker-discuss@python.org https://mail.python.org/mailman/listinfo/tracker-discuss Code of Conduct: https://www.python.org/psf/codeofconduct/
