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

build(futures): Swap package names for futures 0.1 and 0.3 #1362

Merged
merged 4 commits into from
Aug 3, 2022

Conversation

tobias-wilfert
Copy link
Contributor

@tobias-wilfert tobias-wilfert commented Jul 29, 2022

As part of the effort to future-proof Relay as outlined here #1374 the goal is to use the standard Futures instead of the older version of the crate. With this effort currently under way this PR refactors the system by changing futures to futures01 and futures03 to futures. As such it builds further on the changes made here #1374.

#skip-changelog

@tobias-wilfert tobias-wilfert self-assigned this Jul 29, 2022
@flub
Copy link
Contributor

flub commented Jul 29, 2022

This PR is a bit tricky to review since it includes the changes from your other PR. The way to do this in github is to select the branch of your other PR as "base", you can click "edit" to the right of the PR title to do this. That way only the changes from this PR will show up in the diff.

Once your other PR is merged to master github will automatically switch the base of this PR back to "master" and you can safely merge this PR.

@tobias-wilfert tobias-wilfert changed the base branch from master to tobias-wilfert/future-proofing-relay August 1, 2022 05:53
Base automatically changed from tobias-wilfert/future-proofing-relay to master August 1, 2022 09:09
@tobias-wilfert tobias-wilfert force-pushed the tobias-wilfert/refactor-futures branch from c8ef372 to cffe263 Compare August 1, 2022 09:37
@jan-auer jan-auer changed the title ref(futures): Changed default Futures build(futures): Swap package names for futures 0.1 and 0.3 Aug 1, 2022
@tobias-wilfert tobias-wilfert marked this pull request as ready for review August 1, 2022 09:43
@tobias-wilfert tobias-wilfert requested a review from a team as a code owner August 1, 2022 09:43
@tobias-wilfert tobias-wilfert requested a review from a team August 1, 2022 09:43
@tobias-wilfert tobias-wilfert merged commit 6f66945 into master Aug 3, 2022
@tobias-wilfert tobias-wilfert deleted the tobias-wilfert/refactor-futures branch August 3, 2022 05:47
@tobias-wilfert tobias-wilfert mentioned this pull request Aug 18, 2022
30 tasks
@HazAT HazAT added this to the Upgrade Tokio in Relay milestone Nov 21, 2022
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

5 participants