On Sat, Mar 30, 2024 at 11:16 PM Noah Misch <n...@leadboat.com> wrote:

> On Tue, Jan 30, 2024 at 01:49:42PM -0700, David G. Johnston wrote:
> > +   The membership grants created by the
> > +   <literal>IN ROLE</literal>, <literal>ROLE</literal>, and
> <literal>ADMIN</literal>
> > +   clauses have the role executing this command as the grantee.
>
> This should be s/grantee/grantor/, right?
>

Correct.


> > +   If unspecified when create a new role membership this defaults to
> > +   the inheritance attribute of the role being added.
>
> Need s/create/creating/.  Missing comma.  While phrase "role being added"
> isn't wrong, the rest of the paragraph uses "member" and uses "granted
> role"
> to refer to the member-of role.  Putting those together:
>
>   If unspecified when creating a new role membership, this defaults to the
>   inheritance attribute of the new member.
>

I like that better.

David J.

Reply via email to