You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
method is expected to return Promise<T | null>.
However, if you check the yarn build generated lib/src/BaseFirestoreRepository.d.ts, we will see Promise<T>.
This is very unfortunate because it is giving up the benefit of the type itself.
The likely cause is that strictNullChecks has been false.
I tried to build with strictNullChecks set to true, but I got 249 errors and found that I had to change a lot of this library.
Because of this situation, I think it is better to let the owner decide or fix the problem.
The text was updated successfully, but these errors were encountered:
For example,
fireorm/src/BaseFirestoreRepository.ts
Lines 23 to 28 in 2a98b9a
Promise<T | null>
.However, if you check the
yarn build
generatedlib/src/BaseFirestoreRepository.d.ts
, we will seePromise<T>
.This is very unfortunate because it is giving up the benefit of the type itself.
The likely cause is that
strictNullChecks
has beenfalse
.I tried to build with
strictNullChecks
set to true, but I got 249 errors and found that I had to change a lot of this library.Because of this situation, I think it is better to let the owner decide or fix the problem.
The text was updated successfully, but these errors were encountered: