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

vici: add asynchronous session in python module #632

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jeanseb6wind
Copy link

Add a new Python 3.6 package allowing vici to send asynchronous
requests. This feature is only available in Python 3 so we add a
condition in the setup.py in order to not break the Python 2 package.

@lgtm-com
Copy link

lgtm-com bot commented Sep 28, 2021

This pull request introduces 3 alerts when merging d5167d8 into 1b21a00 - view on LGTM.com

new alerts:

  • 2 for Suspicious unused loop iteration variable
  • 1 for Unused import

@Thermi
Copy link
Contributor

Thermi commented Sep 28, 2021

You missed the opportunity to call it avinci

@jeanseb6wind
Copy link
Author

jeanseb6wind commented Sep 30, 2021

Hello,
Can someone take a look at this PR ?
We are using this code in production and it works great.

Thanks

Add a new Python 3.6 package allowing vici to send asynchronous
requests. This feature is only available in Python 3 so we add a
condition in the setup.py in order to not break the Python 2 package.
@ismaillatona
Copy link

It's been 4 months. Looks like you could make a fork project. I'd suggest you name it Aviici ;)

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.

None yet

3 participants