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

chore: refactor dcutr and gossipsub tests to use tokio instead #5662

Merged
merged 3 commits into from
Nov 21, 2024

Conversation

kamuik16
Copy link
Contributor

@kamuik16 kamuik16 commented Nov 5, 2024

Description

ref #4449

Refactored dcutr and gossipsub tests to use tokio instead of async-std.

Copy link
Member

@jxs jxs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this! Looks good to me, but can we split this into two PR's for each behaviour?

@kamuik16
Copy link
Contributor Author

kamuik16 commented Nov 6, 2024

Thanks for this! Looks good to me, but can we split this into two PR's for each behaviour?

Hey, for this one do you mind keeping this one, I'll make sure to make separate PRs for each behavior from next time.

@kamuik16 kamuik16 requested a review from jxs November 6, 2024 06:42
@jxs jxs changed the title chore: refactor dcutr and gossipsub tests chore: refactor dcutr and gossipsub tests to use tokio instead Nov 21, 2024
jxs
jxs previously approved these changes Nov 21, 2024
Copy link
Member

@jxs jxs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jxs jxs added the send-it label Nov 21, 2024
@mergify mergify bot dismissed jxs’s stale review November 21, 2024 12:35

Approvals have been dismissed because the PR was updated after the send-it label was applied.

@kamuik16
Copy link
Contributor Author

LGTM

aah, the mergify couldn't merge becuase the branch wasn't updated so I merge the master into this one. Seems like because of it the approval was dismissed, could you approve it once again?

@kamuik16 kamuik16 requested a review from jxs November 21, 2024 12:38
@jxs
Copy link
Member

jxs commented Nov 21, 2024

LGTM

aah, the mergify couldn't merge becuase the branch wasn't updated so I merge the master into this one. Seems like because of it the approval was dismissed, could you approve it once again?

no need to do that, mergify does it automatically :)

@mergify mergify bot merged commit 13b9ea2 into libp2p:master Nov 21, 2024
70 of 71 checks passed
@kamuik16 kamuik16 deleted the tests-dcutr-gossipsub branch November 21, 2024 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants