beanie icon indicating copy to clipboard operation
beanie copied to clipboard

Type hint overall improving

Open CAPITAINMARVEL opened this issue 9 months ago • 12 comments

Fix correct return type for query update operation, fix type hint for sort, correct return type for document update and removed awaitable making it not able to be used in asyncio task

CAPITAINMARVEL avatar May 06 '24 14:05 CAPITAINMARVEL

for expression field it can allow to define the type like inc, only taking number for inc operation, taking any for set ? for current_time maybe taking only datetime or any affiliate things ?

CAPITAINMARVEL avatar May 06 '24 14:05 CAPITAINMARVEL

for expression field it can allow to define the type like inc, only taking number for inc operation, taking any for set ? for current_time maybe taking only datetime or any affiliate things ?

@roman-right which i can add if you think its ok (i think it would be great to take that)

other than that my pull request is normally ok (i tested it with pytest)?

CAPITAINMARVEL avatar May 06 '24 14:05 CAPITAINMARVEL

Hi @CAPITAINMARVEL , How did you test these types? Did you use type checker?

alm0ra avatar May 08 '24 05:05 alm0ra

Hi @CAPITAINMARVEL , How did you test this types? Did you use type checker?

Thanks for the reply i will fix it

CAPITAINMARVEL avatar May 08 '24 08:05 CAPITAINMARVEL

weird there are many sort missing in find

CAPITAINMARVEL avatar May 08 '24 10:05 CAPITAINMARVEL

gotta fix BaseFindBitwiseOperator

CAPITAINMARVEL avatar May 08 '24 14:05 CAPITAINMARVEL

This PR is stale because it has been open 45 days with no activity.

github-actions[bot] avatar Jun 23 '24 02:06 github-actions[bot]

This PR was closed because it has been stalled for 14 days with no activity.

github-actions[bot] avatar Jul 07 '24 02:07 github-actions[bot]

hmm

CAPITAINMARVEL avatar Aug 17 '24 11:08 CAPITAINMARVEL

@CAPITAINMARVEL, please resolve the conflicts.

adeelsohailahmed avatar Aug 23 '24 19:08 adeelsohailahmed

@CAPITAINMARVEL, please resolve the conflicts.

I just did but pre commit didn’t pass ?

CAPITAINMARVEL avatar Aug 23 '24 21:08 CAPITAINMARVEL

Ah mypy error i see

Ill fix the rest tomorrow on computer didn’t know it would trigger the test again

CAPITAINMARVEL avatar Aug 23 '24 21:08 CAPITAINMARVEL