On Tue, 29 Sep 2015 17:26:36 -0400 Ralph Bean <rbean@xxxxxxxxxx> 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:
pgp9fVjqbSI12.pgp
Description: OpenPGP digital signature
_______________________________________________ infrastructure mailing list infrastructure@xxxxxxxxxxxxxxxxxxxxxxx http://lists.fedoraproject.org/postorius/infrastructure@xxxxxxxxxxxxxxxxxxxxxxx