two party pol #125
Merged
two party pol #125
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Merged
…ates to forwarder, holder
bekauz
force-pushed
the
benskey/covenant-two-party-pol
branch
from
dbf3057
to
24a73a8
Compare
go version env var cd and test in same step v3 checkout installing just
test root justfile workspace-optimizer
test ubuntu-latest cleanup unused deps removing cargo comments
contracts dependency on tests try to setup docker on self hosted runner v2 buildx docker debugging ci bump astroport ver
debug artifacts copying over wasms to test folder fix unit tests apk add node debug wasm folder directory weirdness cd & test
cleanup mv-contracts step split jobs run ictests without justfile
fix test copying over astroport contracts unignore astroport wasms self hosted runner apk add docker docker step docker action docker action in separate step
adding docker apk add docker apt docker install docker install convenience script npm & node docker --version revert to ubuntu-latest tar
|
Let me know once you fix the checks, will do the review. |
Art3miX
approved these changes
Nov 9, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
closes #90
merge after #117
changes to holder and liquid pooler contracts are expected
interchaintest (that needs to be cleaned up after holder/liquidpooler changes) covers happy path and ragequit