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

Remove await statements from with db_session blocks #7170

Merged
merged 3 commits into from
Nov 16, 2022

Conversation

drew2a
Copy link
Contributor

@drew2a drew2a commented Nov 15, 2022

This PR fixes #7167 by:

  1. Removing await statements from the with db_session blocks.
  2. Removing async statements for @db_session annotations.

These changes were made for the whole codebase.

Currently PonyORM doesn't support async operations and it is not recommended to use any await statements inside with db_session blocks.

@drew2a drew2a marked this pull request as ready for review November 15, 2022 15:39
@drew2a drew2a requested review from a team and kozlovsky and removed request for a team November 15, 2022 15:39
Copy link
Contributor

@kozlovsky kozlovsky left a comment

Choose a reason for hiding this comment

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

Great finding!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

[Tests] test_channels_endpoint ERROR at teardown of test_get_my_channel_tags_xxx
2 participants