Skip to content

rpc/client: rewrite the WaitForOneEvent helper#7986

Merged
creachadair merged 1 commit intomasterfrom
mjf/adr075-helper
Feb 24, 2022
Merged

rpc/client: rewrite the WaitForOneEvent helper#7986
creachadair merged 1 commit intomasterfrom
mjf/adr075-helper

Conversation

@creachadair
Copy link

@creachadair creachadair commented Feb 24, 2022

This is part of the implementation of ADR 075. This change updates an existing client helper function, used by tests, to use the new Events API instead of subscriptions.

@creachadair creachadair marked this pull request as ready for review February 24, 2022 17:23
@creachadair creachadair force-pushed the mjf/adr075-helper branch 2 times, most recently from 9c7cc04 to fae64c0 Compare February 24, 2022 18:21
@creachadair creachadair merged commit f795d3f into master Feb 24, 2022
@creachadair creachadair deleted the mjf/adr075-helper branch February 24, 2022 19:21
mmsqe pushed a commit to mmsqe/tendermint that referenced this pull request Aug 29, 2022
mmsqe pushed a commit to mmsqe/tendermint that referenced this pull request Aug 29, 2022
mmsqe pushed a commit to mmsqe/tendermint that referenced this pull request Aug 30, 2022
mmsqe pushed a commit to mmsqe/tendermint that referenced this pull request Aug 30, 2022
mmsqe pushed a commit to mmsqe/tendermint that referenced this pull request Aug 30, 2022
mmsqe pushed a commit to mmsqe/tendermint that referenced this pull request Oct 14, 2022
sergio-mena pushed a commit that referenced this pull request Nov 3, 2022
Update usage in tests.

Co-authored-by: M. J. Fromberger <fromberger@interchain.io>
mmsqe added a commit to mmsqe/tendermint that referenced this pull request Dec 9, 2022
…dermint#9550)

Update usage in tests.

Co-authored-by: M. J. Fromberger <fromberger@interchain.io>
mmsqe added a commit to mmsqe/tendermint that referenced this pull request Dec 9, 2022
…dermint#9550)

Update usage in tests.

Co-authored-by: M. J. Fromberger <fromberger@interchain.io>
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.

2 participants