Skip to content

e2e: Fix hashing for app + Fix logic of TestApp_Hash (backport #8229)#8236

Merged
tychoish merged 2 commits intov0.35.xfrom
mergify/bp/v0.35.x/pr-8229
Apr 1, 2022
Merged

e2e: Fix hashing for app + Fix logic of TestApp_Hash (backport #8229)#8236
tychoish merged 2 commits intov0.35.xfrom
mergify/bp/v0.35.x/pr-8229

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Mar 31, 2022

This is an automatic backport of pull request #8229 done by Mergify.


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

* Fix hashing of e2e App

* Fix TestApp_Hash

* CaMeL

* Update test/e2e/app/state.go

Co-authored-by: M. J. Fromberger <fromberger@interchain.io>

* for-->Eventually + if-->require

* Update test/e2e/tests/app_test.go

Co-authored-by: Sam Kleinman <garen@tychoish.com>

* fix lint

Co-authored-by: M. J. Fromberger <fromberger@interchain.io>
Co-authored-by: Sam Kleinman <garen@tychoish.com>
(cherry picked from commit 4a504c0)
@sergio-mena sergio-mena added the S:automerge Automatically merge PR when requirements pass label Mar 31, 2022
@tychoish tychoish merged commit 524d3ce into v0.35.x Apr 1, 2022
@tychoish tychoish deleted the mergify/bp/v0.35.x/pr-8229 branch April 1, 2022 15:12
@sergio-mena
Copy link
Contributor

Thanks @tychoish for merging

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S:automerge Automatically merge PR when requirements pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants