Why can't the content True == successful delete?


On Mon, Apr 7, 2014 at 5:14 PM, Alex Zylman <notifications@github.com>wrote:

> In salt/_modules/hk_collaborator.py:
>
> > +
> > +def list(token, app):
> > +    '''
> > +    List all collaborators for a Heroku app. See https://devcenter.heroku.com/articles/platform-api-reference#collaborator-list.
> > +
> > +    CLI Example:
> > +
> > +    .. code-block:: bash
> > +
> > +        sudo salt-call --local hk_collaborators.list <token> <app>
> > +    '''
> > +    r = requests.get('https://api.heroku.com/apps/{}/collaborators'.format(app),
> > +                     headers={'Authorization': 'Bearer ' + token,
> > +                              'Accept': 'application/vnd.heroku+json; version=3'})
> > +    if not r.ok:
> > +        log.error('Error making Heroku API request: {}'.format(r))
>
> Looks real good! Can we get the consistent return values, too? One still
> returns True instead of content.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/Clever/salt-org/pull/3/files#r11368534>
> .
>