Simon Garner

Results 62 comments of Simon Garner

I just upgraded a project from @nestjs/graphql v8 to v12 (with Apollo driver) and noticed the behaviour of `defaultValue` changed. The declaration : ```ts @ArgsType() class MyArgs { @Field(() =>...

@smolinari Sorry, but that's just not right. You're conflating two different concepts. Setting an argument to `null` and omitting an argument entirely are not the same thing. > Not sending...

> But, they are considered the same in GraphQL. I'm not sure why you say this? I've already pointed to the sections of the specification which explain otherwise, please refer...

> Where do these rules say that an omitted value is not considered as "null" to GraphQL? They don't - but nor have I seen anything that agrees with you...

> There is no default in the TS code, so there is no way you are getting `= "bar"` in the DSL. It doesn't matter what default is set in...

This happens all the time for me due to automated dependency updates (by Renovate) and infrequent releases. It would be great to simply have an option to disable automatic generation...

@tommy-mitchell I still want `np` create the draft release though, I just don't want it to pre-populate the notes in the body. Of course, yes, I could create the release...

> Makes sense. Would you mind opening another issue suggesting that? If you have any ideas in mind for the option name, that’d be helpful too. Sure, here's a new...

TypeORM has [`RawSqlResultsToEntityTransformer`](https://github.com/typeorm/typeorm/blob/master/src/query-builder/transformer/RawSqlResultsToEntityTransformer.ts#L18) which can help with this mapping. The catch is that you need to pass a `QueryExpressionMap` instance to it which might be difficult to build manually. However...

> Nice, well that's new! Thanks for the heads up. Nothing has changed? There has always been the ability to filter in events by whitelisting particular labels. But that's not...