Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix return type from document update #1030

Merged
merged 4 commits into from
Oct 4, 2024

Conversation

CAPITAINMARVEL
Copy link
Contributor

Argument of type "Awaitable[Player]" cannot be assigned to parameter "coro" of type "_CoroutineLike[_T@create_task]" in function "create_task"
"Awaitable[Player]" is not assignable to "Coroutine[Any, Any, _T@create_task]"PylancereportArgumentType

Beanie incorrectly use Awaitable for document set which make raising error on using asyncio create_task

@CAPITAINMARVEL CAPITAINMARVEL requested a review from a team September 30, 2024 08:56
@CAPITAINMARVEL CAPITAINMARVEL self-assigned this Sep 30, 2024
Riverfount
Riverfount previously approved these changes Sep 30, 2024
Copy link

@Riverfount Riverfount left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

beanie/odm/documents.py Outdated Show resolved Hide resolved
beanie/odm/documents.py Outdated Show resolved Hide resolved
fix type hint
LawMixer
LawMixer previously approved these changes Oct 3, 2024
staticxterm
staticxterm previously approved these changes Oct 3, 2024
Copy link

@staticxterm staticxterm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good work, thank you!

Copy link

@Riverfount Riverfount left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@CAPITAINMARVEL CAPITAINMARVEL merged commit 4318aa9 into BeanieODM:main Oct 4, 2024
56 checks passed
@CAPITAINMARVEL CAPITAINMARVEL deleted the fix_return_type branch October 4, 2024 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants