On Tue, 29 Sep 2015 17:26:36 -0400
Ralph Bean <[email protected]> wrote:

> On Tue, Sep 29, 2015 at 01:45:15PM -0600, Tim Flink wrote:
> > > It looks like this function used to return False if an invalid
> > > playbook_file was passed to it, but now it will raise a
> > > KeyError.  Is that intentional?  I see there is a test case for
> > > it, but the `rbac_playbook` function doesn't handle it.
> > 
> > KeyError now handled and test case added, thanks.
> > 
> > Tim
> 
> Looks good now.  +1, and thanks!

Yeah, seems good to me too. ;) 

kevin

Attachment: pgpkwkLtGI5Rl.pgp
Description: OpenPGP digital signature

_______________________________________________
infrastructure mailing list
[email protected]
http://lists.fedoraproject.org/postorius/[email protected]

Reply via email to