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.

Sign up for GitHub

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

Jump to bottom

feat: add thorchain support #1219

Merged
merged 40 commits into from
Aug 22, 2024
Merged

feat: add thorchain support #1219

merged 40 commits into from
Aug 22, 2024

Conversation

misko9
Copy link
Contributor

@misko9 misko9 commented Aug 15, 2024 •
edited
Loading

Adds thorchain support.
Adds example tests for thorchain (sim and hard fork).

misko9 and others added 30 commits July 19, 2024 14:53
@misko9
initial thorchain addition, configs setup, still need genesis update
ffe6544
@misko9
wip: thorchain genesis
9116cec
@misko9
thorchain running
ceba377
@misko9
thorchain and bifrost working together
277335b
@misko9
add savers, swaps, and saver eject example case for gaia
3e6981b
@misko9
arb wip, all other sim tests passing
49eafc3
@misko9
arb code in, needs testing with >1 pool
c0febe8
@misko9
add ethereum to thorchain test and start cleaning up
cbc6d2d
@misko9
clean up savers/arb features
50a6017
@misko9
clean up swap feature
699ebf5
@misko9
cleanup of saver eject and ragnarok features
de5eb93
@misko9
fix: second+ saver eject now works
2fcd796
@misko9
Add back arbing and eth->gaia swap
ac8aac9
@misko9
wip, add utxo support, SendFundsWithNote() still needed
cfc4091
@agouin
hardfork wip
e1c5671
@agouin
start with genesis contents
de8797a
@agouin
progress further
a166e88
@misko9
wip: removed non-BTC chains temporarily, BTC's SendFundsWithNote succ…
95520c5 
…eeds.
@misko9
thorchain<->btc dual lp working
69977a6
@misko9
btc and bch looking good
3fa9983
@misko9
ltc working, clean up logging, fix coins funded on each chain, and fi…
ea0a2a0 
…x utxo send amount
@misko9
utxo chains fully operational
34f5287
@misko9
fix ether type
4346aae
@misko9
add some protections around utxo node wallet usage
e39bd9a
@misko9
send utxo change back to sender instead of a change address
ad4a0a7
@misko9
Run tests in parallel
5daf638
@misko9
clean up utxo test
2dbbf78
@misko9
fmt/lint
f08f494
@misko9
More cleanup
7508c67
@misko9
increase time for bifrost to initialize
d1641df
@vercel Vercel
Copy link

vercel bot commented Aug 15, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
interchaintest-docs ⬜️ Ignored (Inspect) Visit Preview Aug 22, 2024 0:07am

@misko9 misko9 changed the title feat: add utxo chain and thorchain support feat: add thorchain support Aug 15, 2024
@misko9 misko9 marked this pull request as ready for review August 21, 2024 22:50
@misko9 misko9 requested a review from a team as a code owner August 21, 2024 22:50
Reecepbcups
Reecepbcups reviewed Aug 22, 2024
View reviewed changes
chain/thorchain/address.go Show resolved Hide resolved
ibc/types.go Show resolved Hide resolved
interchain.go Show resolved Hide resolved
chain/thorchain/module_bank.go Outdated Show resolved Hide resolved
chain/thorchain/wallet.go Show resolved Hide resolved
examples/thorchain/chainspec_exochains.go Show resolved Hide resolved
examples/thorchain/contracts/lib/forge-std/.github/workflows/ci.yml Show resolved Hide resolved
chain/thorchain/thorchain.go Show resolved Hide resolved
This was referenced Aug 22, 2024
Open
Open
Reecepbcups
Reecepbcups approved these changes Aug 22, 2024
View reviewed changes
Copy link
Member

@Reecepbcups Reecepbcups left a comment

Choose a reason for hiding this comment

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

proactive approval

@Reecepbcups
Copy link
Member

🚢

@Reecepbcups Reecepbcups merged commit 2f1634e into main Aug 22, 2024
22 checks passed
@Reecepbcups Reecepbcups deleted the steve/thorchain branch August 22, 2024 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Reecepbcups Reecepbcups Reecepbcups approved these changes

@vimystic vimystic Awaiting requested review from vimystic vimystic is a code owner automatically assigned from strangelove-ventures/interchaintest-maintainers

@agouin agouin Awaiting requested review from agouin

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@misko9 @Reecepbcups @agouin