feat!: add XCM weight trader and update integration tests #475
Conversation
|
Crate versions that have not been updated:
Crate versions that have been updated:
Runtime version has not been increased. |
enthusiastmartin
left a comment
enthusiastmartin
left a comment
There was a problem hiding this comment.
great. i was not aware that the trader impl is not actually there in the config.
btw, it would be great to add additioan integration tests ( can be part of another PR) as to fix #469 .
| MultiLocation::new( | ||
| 1, | ||
| X2( | ||
| Junction::Parachain(HYDRA_PARA_ID), |
There was a problem hiding this comment.
nitpick: i think this needs a reformat ?
There was a problem hiding this comment.
running cargo fmt does nothing. What in particular don't you like?
Codecov ReportBase: 85.77% // Head: 85.77% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## polkadot-v0.9.29 #475 +/- ##
=================================================
Coverage 85.77% 85.77%
=================================================
Files 8 8
Lines 1005 1005
=================================================
Hits 862 862
Misses 143 143 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
…-node into apopiak/weight-trader
apopiak
changed the title
…-node into apopiak/weight-trader
Adds XCM weight trader. Analogous to galacticcouncil/Basilisk-node#443
Description
Related Issue
Motivation and Context
How Has This Been Tested?
Checklist: