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 #8

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

Fix #8

wants to merge 1 commit into from

Conversation

populated
Copy link
Contributor

Comment out rust_requests for now, as it may cause installation errors for people without Rust/Cargo installed on their system.

Comment out `rust_requests` for now, as it may cause installation errors for people without Rust/Cargo installed on their system.
@cw-e
Copy link

cw-e commented Apr 25, 2024

I had to install rust due to this thanks for this <3

@cw-e
Copy link

cw-e commented Apr 25, 2024

You also need to have an event loop policy like this:
if you are on windows... for some reason aiodns bugs with an error message

asyncio.set_event_loop_policy(asyncio.WindowsSelectorEventLoopPolicy())

@populated
Copy link
Contributor Author

You also need to have an event loop policy like this: if you are on windows... for some reason aiodns bugs with an error message

asyncio.set_event_loop_policy(asyncio.WindowsSelectorEventLoopPolicy())

I'll look into this!

@populated
Copy link
Contributor Author

I had to install rust due to this thanks for this <3

Sadly, I know. I got in contact with the cop. I'm just waiting for a response. rust_requests is going to be used for a separate branch of disdick that will be faster than the current branch.

@cop-discord
Copy link
Owner

rust_requests can be removed

@cop-discord
Copy link
Owner

rust requests has been removed but we also use fast string match which is also using rust so its up to you whether you want to comment that out aswell. i will make it an optional import aka only imported inside of the functions it is used in

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.

3 participants