search

bastion-rs / agnostik

Executor Agnostic Runtime that can run your futures with your favourite Executor
https://docs.rs/agnostik
Apache License 2.0
143 stars 11 forks source link

Remove default. #27

Closed dvc94ch closed 3 years ago

dvc94ch commented 3 years ago

To really be executor agnostic, I think there shouldn't be a default executor enabled. The other issue is that if you enable an executor, you also need to set no-default-features and in the library case you don't want any executor so you also need to disable the default. Considering this, it's likely a good idea to not enable an executor by default.

Stupremee commented 3 years ago

Firstly, thank you for your PRs I will have a look at them now. I will also close this PR and implement this as part of #21