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

Move response futures into their own modules #130

Merged
merged 1 commit into from
Aug 5, 2021
Merged

Conversation

davidpdrsn
Copy link
Member

It cleans up the docs to have the futures in their own modules as users
are unlikely to look at them. Also matches the pattern used in tower
https://docs.rs/tower/0.4.8/tower/util/future/index.html.

Added re-exports to the old locations so its not a breaking change.

It cleans up the docs to have the futures in their own modules as users
are unlikely to look at them. Also matches the pattern used in tower
https://docs.rs/tower/0.4.8/tower/util/future/index.html.

Added re-exports to the old locations so its not a breaking change.
@davidpdrsn davidpdrsn added the C-cleanup Category: PRs that clean code up or issues documenting cleanup. label Aug 5, 2021
@davidpdrsn davidpdrsn merged commit d4ce90e into main Aug 5, 2021
@davidpdrsn davidpdrsn deleted the move-futures branch August 5, 2021 23:15
davidpdrsn added a commit that referenced this pull request Aug 5, 2021
These types were moved around in
#130 but re-export from their old
location for backwards compatibility.

This removes the re-exports.
davidpdrsn added a commit that referenced this pull request Aug 7, 2021
These types were moved around in
#130 but re-export from their old
location for backwards compatibility.

This removes the re-exports.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-cleanup Category: PRs that clean code up or issues documenting cleanup.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant