Support non-nullable list items as before

[newellista]

Expected Behaviour

func (g *GraphQL) Query_users(ctx context.Context) ([]User, error)

Actual Behavior

func (g *GraphQL) Query_users(ctx context.Context) ([]*User, error)

Minimal graphql.schema and models to reproduce

type Query {
  users: [Users]
}

Notes:

Before #191 this worked correctly. However, after that PR was merged, it always generates the optional form when returning a slice.

I would expect

type Query {
  users: [Users]!
}

to return

func (g *GraphQL) Query_users(ctx context.Context) ([]*User, error)

while

type Query {
  users: [Users]
}

should return:

func (g *GraphQL) Query_users(ctx context.Context) ([]User, error)

[vektah]

Did you try a required list of required elements:

type Query {
    user: [Users!]!
}

What you have [Users]! is a required list of nullable elements

read more