remove usages of futures::select! in the repo with futures_concurrency
Open
Nobody has claimed this yet.
good first issue
linting
- Dominant language
- Rust
- Stars
- 1.9k
- Forks
- 238
- Avg merge
- 1d 15h
- Merged PRs (30d)
- 100
Description
select! has subtle rules you need to get right, and we really would like to push people towards using things in futures_concurrency which are nicer to use.
We should remove the exiting usages in the repo and replace them, then disallow select! in clippy. Maybe a good first step is to disallow select! and require overriding for the existing usages so we can not mislead people into using select!
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.