Gabriel Rohden

@tthornton3-chwy I agree with that and would prefer to lean towards function comps for max compatibility 🤔 I guess I went with the class approach cuz that was supposed to...

> Thank you all guys! The PR is ready to review (#1521) > > > Sounds about right. Nevertheless @leanmazzu I'm highly intrigued with what your use cases are, I...

Once I put `strict: true` in TS config it infers properly... this might be another type of issue that you get when dealing with null types on a codebase that...

@xegulon did you try `strict` + `strictNullChecks`?

just launching the idea for who might be thinking on fixing drizzle to not care about nulls depending on the flag: ```ts type IsStrictNullEnabled = null extends number ? false...

Tried to debug it, here's what I found * My global yarn is v1, my project is using v4/v3, patch package was using v1 * I've tried to fix that...

I'll link this issue https://github.com/yarnpkg/berry/issues/5802 But I think that ultimately, the issue is that patch package should copy the "packageManager" field to the temp `package.json` so that yarn (corepack?) doesn't...

Here's a patch that made it work for me ```diff Index: src/makePatch.ts IDEA additional info: Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP UTF-8 =================================================================== diff --git a/src/makePatch.ts b/src/makePatch.ts --- a/src/makePatch.ts (revision c7c63bf80b3c6b8640b933e20229121b4edfc100) +++ b/src/makePatch.ts (date...

I swear this `use-npm` flag worked before, now it just vanished? Anyway, I think we'll need support for it in https://github.com/ds300/patch-package/blob/bd2e9a49d884516199079add143c1649541e8efe/src/index.ts#L87 since this one is not going to be solved...

@jerelmiller what would be the problem on clients (devs) doing something like this? ```ts export function useQuery< TData = any, TVariables extends OperationVariables = OperationVariables, >( query: TypedDocumentString, options?: QueryHookOptions...